Words by Török Edvin [Sat, Mar 03, 2007 at 11:40:52PM +0200]: > On 3/2/07, Jose Celestino <[EMAIL PROTECTED]> wrote: > >Words by Jose Celestino [Sat, Feb 17, 2007 at 01:24:10AM +0000]: > >> Hi, > >> > >> stepped into an infinite loop on freshclam when there's no permissions > >> for daily.inc and daily.inc/*. > >> > >> I know the permissions are an install problem but it may happen and the > >> loop is a major CPU chewer: > >> > >> root 232 0.0 0.0 1928 652 ? S 2006 0:46 /usr/sbin/cron > >> root 4915 0.0 0.0 2040 724 ? S 13:44 0:00 \_ /USR/SBIN/CRON > >> qscand 4916 73.9 0.0 2892 848 ? R 13:44 421:12 \_ > >/ptmail/av/bin/freshclam > >> root 4918 0.0 0.0 1524 300 ? S 13:44 0:00 \_ bin/qmail-inject -a -- root > >> qmailq 4922 0.0 0.0 1488 288 ? S 13:44 0:00 \_ bin/qmail-queue > >> > >> Attached is a patch for shared/misc.c that fixes this (haven't tested > >> it though) and a strace dump of the looping process. > >> > > > >Anyone considered merging this patch (or solving this any other way)? Been > >bitten a couple for times lately with reinstalls leaving the old owner > >on the directory. > > Doesn't this commit fix the problem: > > r2842 | tkojm | 2007-02-22 20:13:13 +0200 (Thu, 22 Feb 2007) | 2 lines > > drop rmdirs() and use cli_rmdirs() instead
Ah, you're right :) -- Jose Celestino ---------------------------------------------------------------- http://www.msversus.org/ ; http://techp.org/petition/show/1 http://www.vinc17.org/noswpat.en.html ---------------------------------------------------------------- "And on the trillionth day, Man created Gods." -- Thomas D. Pate _______________________________________________ http://lurker.clamav.net/list/clamav-devel.html Please submit your patches to our Bugzilla: http://bugs.clamav.net