No, your patch doesn't fix the issue, I still get the same segfault.
Reverting back to the previous patch and everything works fine.

Upon inspection of the code it is this bit that causes the segfault:
        while (i-- && isspace(buf[i]))
                buf[i] = 0;

Its because of the white space check in the while(), it negates 1 from i
and then does a check on it (so i is out of bounds of buf), re arranging
this so the check it first solves the segfault but also causes it to put
lots of white space in the device and partition names.

The patch I posted previously works perfectly, although the code is a little 
jumbled, I have cleaned it and attached it to this.
This works perfectly as my previous one did also.

** Attachment added: "Jmicron Name fix patch"
   http://launchpadlibrarian.net/24147483/05_jmicron-name-fix.patch

-- 
JMicron RAID controller segfaults when activating dmraid-rc15 (jaunty)
https://bugs.launchpad.net/bugs/344215
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

Reply via email to