Bug in mdadm (2.5.6-8) postinst script

"Jørn V. Christensen" mail at mettle.dk
Sat Mar 3 23:33:56 CET 2007


Hi,

I am writing to you because you are listed as the maintainer of the 
mdadm package in Debian. I have found (and fixed?) a bug in the post 
installation script for mdadm 2.5.6-8.

Basically what happened, was that sed complained about a misstated command:
    sed: -e expression #1, char 30: unknown option to `s'

I investigated the error and found that the it happened while sed was 
trying to update the mail addresses. The problem is that I have listed 
two mail addresses (separated by a comma) and the sed command delimiter 
character was also chosen to be commas. Thus the resulting sed command 
would have 4 commas instead of 3.

The fix: Changing the sed delimiter character to / instead of comma.

73c73
<         sed -i -e "s,^MAILADDR.*,MAILADDR $MAILADDR," $CONFIG
---
 >         sed -i -e "s/^MAILADDR.*/MAILADDR $MAILADDR/" $CONFIG

The bug is rather old (several months) but I did not do anything about 
it because I thought you (as the maintainer) would see the bug and make 
a fix. I did not realize until now that it was because I have two 
addresses listed instead of one.

Anyway... hope it is useful to you...

Best regards
Jørn




More information about the pkg-mdadm-devel mailing list