Bug 4768 - udev rule use bash
Summary: udev rule use bash
Status: RESOLVED FIXED
Alias: None
Product: Mageia
Classification: Unclassified
Component: RPM Packages (show other bugs)
Version: Cauldron
Hardware: All Linux
Priority: Normal normal
Target Milestone: ---
Assignee: Thomas Backlund
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-03-02 08:53 CET by Thierry Vignaud
Modified: 2012-09-05 16:25 CEST (History)
3 users (show)

See Also:
Source RPM: mdadm-3.2.3-1.mga2
CVE:
Status comment:


Attachments
Fix udev rule (833 bytes, patch)
2012-03-02 10:12 CET, Thierry Vignaud
Details | Diff

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


Note You need to log in before you can comment on or make changes to this bug.