Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=487665 Bill Nottingham <notting@xxxxxxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |ajax@xxxxxxxxxx, | |davidz@xxxxxxxxxx, | |katzj@xxxxxxxxxx --- Comment #6 from Bill Nottingham <notting@xxxxxxxxxx> 2009-02-27 16:25:23 EDT --- OK, so looking this over, this doesn't seem right. 1) It implements a critical path in the boot process... in perl. 2) It's needlessly baroque for what it does. You set up a perl daemon to listen to udev for events, look for events of a certain type, and then run actions. Why not just write udev rules that run actions on certain events, and cut out the daemon middleman? 3) It's set to start bluetooth only if it's configured to start. Which means it's likely to have *already started*, and therefore doesn't decrease the running daemon footprint at all. 4) It drops stuff into predictable file names in /tmp, which is a security hole. (Presumably, that's leftover debugging) CC'ing some other people who may have ideas on how to better implement this. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug. _______________________________________________ Fedora-package-review mailing list Fedora-package-review@xxxxxxxxxx http://www.redhat.com/mailman/listinfo/fedora-package-review