Bug 4768

Summary: udev rule use bash
Product: Mageia Reporter: Thierry Vignaud <thierry.vignaud>
Component: RPM PackagesAssignee: Thomas Backlund <tmb>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: Normal CC: alien, davidwhodgins, mageia
Version: Cauldron   
Target Milestone: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Source RPM: mdadm-3.2.3-1.mga2 CVE:
Status comment:
Attachments: Fix udev rule

Description Thierry Vignaud 2012-03-02 08:53:52 CET
mdadm's udev rule use bash:
/lib/udev/rules.d/65-md-incremental.rules:IMPORT{program}="/bin/bash -c '[ -f /dev/.in_sysinit ] && echo in_sysinit=1 || :'"

However it's used in installer where we don't have bash.
Please make it work w/o bash.
Thanks.
Thierry Vignaud 2012-03-02 08:54:00 CET

Blocks: (none) => 4724

Comment 1 Thierry Vignaud 2012-03-02 10:12:07 CET
Created attachment 1670 [details]
Fix udev rule

Alternatively I can use this patch instead but I think it would be faster for udev to fork sh than bloated bash
Comment 2 AL13N 2012-03-02 21:12:40 CET
i checked, but i didn't see any bashisms, so i don't see any problems having it start with /bin/sh

CC: (none) => alien

Thierry Vignaud 2012-03-05 18:18:01 CET

Blocks: 4724 => (none)

Comment 3 Marja Van Waes 2012-05-26 13:03:38 CEST
Hi,

This bug was filed against cauldron, but we do not have cauldron at the moment.

Please report whether this bug is still valid for Mageia 2.

Thanks :)

Cheers,
marja

Keywords: (none) => NEEDINFO

Comment 4 Dave Hodgins 2012-05-27 23:27:29 CEST
grep bash /lib/udev/rules.d/65-md-incremental.rules
IMPORT{program}="/bin/bash -c '[ -f /dev/.in_sysinit ] && echo in_sysinit=1 || :'"

Still applies to both Mageia 2 and Cauldron.

CC: (none) => davidwhodgins

Dave Hodgins 2012-05-27 23:28:00 CEST

Keywords: NEEDINFO => (none)
Version: Cauldron => 2

Comment 5 AL13N 2012-05-28 09:22:00 CEST
but... since we have bash, we should only fix this in cauldron
Comment 6 Thierry Vignaud 2012-05-29 12:09:51 CEST
Indeed it's only for cauldron.
That would enable me to remove this workaround when building installer stage2:

http://svnweb.mageia.org/soft/drakx/trunk/perl-install/install/share/list.xml?r1=3166&r2=3171&sortby=date

CC: (none) => mageia
Version: 2 => Cauldron

Comment 7 Thierry Vignaud 2012-09-05 16:25:29 CEST
I fixed it

Status: NEW => RESOLVED
Resolution: (none) => FIXED