Please read this before reporting a bug:
https://wiki.archlinux.org/title/Bug_reporting_guidelines
Do NOT report bugs when a package is just outdated, or it is in the AUR. Use the 'flag out of date' link on the package page, or the Mailing List.
REPEAT: Do NOT report bugs for outdated packages!
https://wiki.archlinux.org/title/Bug_reporting_guidelines
Do NOT report bugs when a package is just outdated, or it is in the AUR. Use the 'flag out of date' link on the package page, or the Mailing List.
REPEAT: Do NOT report bugs for outdated packages!
FS#13263 - [initscripts] Fix array count/walk in rc.shutdown and rc.single
Attached to Project:
Arch Linux
Opened by Gerardo Exequiel Pozzi (djgera) - Friday, 13 February 2009, 21:02 GMT
Last edited by Thomas Bächler (brain0) - Sunday, 07 June 2009, 19:35 GMT
Opened by Gerardo Exequiel Pozzi (djgera) - Friday, 13 February 2009, 21:02 GMT
Last edited by Thomas Bächler (brain0) - Sunday, 07 June 2009, 19:35 GMT
|
DetailsHi,
When counting elements of DAEMONS array these are counted OK, but when this is walked it start from this count, then visit an inexistent element. Then should be count minus one, (zero based arrays). Trivial fix is attached for rc.single and rc.shutdown There are three attachd files: initscripts-2009.01-1.rcshutdown.arraycount.fix.patch This is for initscripts-2009.01-1 (applied ok with latest commit of initscripts-2009.01-1.rcsingle.arraycount.fix.patch (same here) initscripts-2009.01-1-plus-FS12978.rcsingle.arraycount.fix.patch and this is in the case if this Thanks. |
This task depends upon
Closed by Thomas Bächler (brain0)
Sunday, 07 June 2009, 19:35 GMT
Reason for closing: Fixed
Additional comments about closing: fixed in git
Sunday, 07 June 2009, 19:35 GMT
Reason for closing: Fixed
Additional comments about closing: fixed in git
Comment by Gerardo Exequiel Pozzi (djgera) -
Friday, 13 February 2009, 21:05 GMT
Oooh writing FS#nnnnn is automagically converted to link, cool :P
Comment by Aaron Griffin (phrakture) -
Friday, 13 February 2009, 21:25 GMT
- Field changed: Status (Unconfirmed → Assigned)
- Task assigned to Aaron Griffin (phrakture), Thomas Bächler (brain0)
Patch looks ok - need to confirm the logic here, though
Comment by Thomas Bächler (brain0) -
Sunday, 07 June 2009, 14:35 GMT
Just checked it, it is correct, commiting to git.
Comment by Gerardo Exequiel Pozzi (djgera) -
Sunday, 07 June 2009, 18:45 GMT
Thanks Thomas, requesting closure...
initscripts-2009.01-1.rcshutd...