gmzang / maczfs

Automatically exported from code.google.com/p/maczfs
Other
0 stars 0 forks source link

panic-decode should handle spaces in paths #81

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
What steps will reproduce the problem?
1. mount /Volumes/OS X 10.6 (with panics)
2. try to tell panic-decode to look for panics there

What is the expected output?
happy panic log

What do you see instead?
various errors resulting from not handling spaces

What version of the product are you using?
"untested"

On what operating system?
os x 10.6.6

my original draft changes which partially cover spaces is here:
http://zfs-macos.googlegroups.com/attach/3fad24ef393567af/panic-decode.changes?v
iew=1&part=3

however it doesn't handle spaces in DebugKit/KERNEL/GDB_KERNEL and those can 
easily happen if you mount multiple versions of the kernel symbols packages - 
from memory you get </Volumes/$disk_name ($increasing_number)>. I'll try to 
provide a more complete patch later, but if i don't, someone should at least 
peel out the quoting related changes from my draft.

Original issue reported on code.google.com by timel...@gmail.com on 8 Feb 2011 at 9:01

GoogleCodeExporter commented 8 years ago
Fixed in 4fdf1d20ba2db29c17ee04b83e22fb1352baf3ee

Original comment by alex.ble...@gmail.com on 16 Feb 2011 at 10:07

GoogleCodeExporter commented 8 years ago
Fixed in 74.1.0 and above

Original comment by alex.ble...@gmail.com on 5 Mar 2011 at 11:55