[Bug 7880] modprobe: already_loaded function should not return true if the state of the module in /proc/modules is not 'Live'

bugzilla at busybox.net bugzilla at busybox.net
Fri Feb 13 15:49:55 UTC 2015


https://bugs.busybox.net/show_bug.cgi?id=7880

--- Comment #5 from Harish <harishjennykn at yahoo.com> 2015-02-13 15:49:54 UTC ---
(In reply to comment #4)
> (In reply to comment #3)
> > Thanks!
> > Fixed in git.
> 
> Nice fix - 
> 
> But one review comment - 
> 
> ret | 1; condition is wrong. 
> it always returns true and already_loaded as success.
> 
> 
> I wish my name was also there in <Signed-off> section !!



Yet again disappointed that my name is not present in signed-off section even
for the review comment fix - 
http://git.busybox.net/busybox/commit/?id=402afe1cc69ea505c9bf82ffe06e51ffecf694df

Even the reviewer's name could have been included along the with the coders
name.

-- 
Configure bugmail: https://bugs.busybox.net/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.


More information about the busybox-cvs mailing list