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.0 +0200
+++ quota-3.12/quotaops.c 2005-05-11 11:58:16.0 +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.0 +0100
+++ quota-3.12/rquota_client.c 2005-05-11 11:58:16.0 +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