Steve, > There appear to be a number of undocumented changes, relative to -4, which > are not documentation fixes. :/ Would you remind removing them and > uploading a -4sarge1 to testing-proposed-updates, or else explain why these > changes are warranted during the freeze?
Please take my apologies. I was interrupted while working on the package and simply forgot to remove some parts of the whole patch I applied. Also I didn't complete the changelog entry. And you probably noticed that stray patch file lying around in the source tree. Argh! I fixed this and am currently uploading -6 which should be okay. It still contains two non-doc patches: diff -ruN tmp/quota-3.12/quotaops.c quota-3.12/quotaops.c --- tmp/quota-3.12/quotaops.c 2005-05-11 11:58:35.000000000 +0200 +++ quota-3.12/quotaops.c 2005-05-11 11:58:16.000000000 +0200 @@ -34,7 +34,7 @@ #ident "$Copyright: (c) 1980, 1990 Regents of the University of California. $" #ident "$Copyright: All rights reserved. $" -#ident "$Id: quotaops.c,v 1.14 2004/05/24 19:39:15 jkar8572 Exp $" +#ident "$Id: quotaops.c,v 1.16 2005/03/31 11:48:02 jkar8572 Exp $" #include <rpc/rpc.h> #include <sys/types.h> @@ -72,13 +72,13 @@ time_t now; time(&now); - if (q->dq_dqb.dqb_bsoftlimit && toqb(q->dq_dqb.dqb_curspace) >= q->dq_dqb.dqb_bsoftlimit) { + if (q->dq_dqb.dqb_bsoftlimit && toqb(q->dq_dqb.dqb_curspace) > q->dq_dqb.dqb_bsoftlimit) { if (!q->dq_dqb.dqb_btime) q->dq_dqb.dqb_btime = now + q->dq_h->qh_info.dqi_bgrace; } else q->dq_dqb.dqb_btime = 0; - if (q->dq_dqb.dqb_isoftlimit && q->dq_dqb.dqb_curinodes >= q->dq_dqb.dqb_isoftlimit) { + if (q->dq_dqb.dqb_isoftlimit && q->dq_dqb.dqb_curinodes > q->dq_dqb.dqb_isoftlimit) { if (!q->dq_dqb.dqb_itime) q->dq_dqb.dqb_itime = now + q->dq_h->qh_info.dqi_igrace; } [...] This one fixes the bug that grace time was set when reaching the limit instead of starting it when exceeding the limit. diff -ruN tmp/quota-3.12/rquota_client.c quota-3.12/rquota_client.c --- tmp/quota-3.12/rquota_client.c 2004-02-12 10:45:14.000000000 +0100 +++ quota-3.12/rquota_client.c 2005-05-11 11:58:16.000000000 +0200 @@ -149,8 +149,10 @@ * Strip off pathname on nfs mounted dir. Ignore entries of any * automounter. */ - if ((pathname = strchr(fsname_tmp, ':')) == (char *)0 || *(pathname + 1) == '(') + if ((pathname = strchr(fsname_tmp, ':')) == (char *)0 || *(pathname + 1) == '(') { + free(fsname_tmp); return -ENOENT; + } *pathname++ = '\0'; This one obviously fixes a memory leak. It's not that serious but I rather have it fixed for sarge too. > Incidentally, I'm puzzled why quota is calling update-inetd at all from the > postrm, since there's no code in the postinst which adds an inetd entry. > That shouldn't be a blocker for sarge, though. You're right. That's why I removed the line completely. Michael -- Michael Meskes Email: Michael at Fam-Meskes dot De, Michael at Meskes dot (De|Com|Net|Org) ICQ: 179140304, AIM/Yahoo: michaelmeskes, Jabber: [EMAIL PROTECTED] Go SF 49ers! Go Rhein Fire! Use Debian GNU/Linux! Use PostgreSQL!
signature.asc
Description: Digital signature