NetBSD Problem Report #18422
Received: (qmail 29920 invoked by uid 605); 26 Sep 2002 04:03:11 -0000
Date: Thu, 26 Sep 2002 13:03:06 +0900 (JST)
Subject: distribute:write_index() is called too early.
>Synopsis: pkgsrc/mail/distribute: write_index() is called too early.
>Arrival-Date: Thu Sep 26 04:04:00 +0000 2002
>Last-Modified: Mon Aug 10 06:35:01 +0000 2015
>Originator: enami tsugutomo
>Release: NetBSD 1.6G
System: NetBSD parity-error.sm.sony.co.jp 1.6G NetBSD 1.6G (PARITY_ERROR.UP.PROF) #315: Wed Sep 4 11:02:52 JST 2002 firstname.lastname@example.org:/a/compile/PARITY_ERROR.UP.PROF i386
The function write_index() in distribute.c (in pkgsrc/mail/distribute)
uses global variable `bodysum' but is called before the check sum
is calculated. As a result, check sum written in INDEX file is always
zero and the program `archive' may barf as follows:
Sep 26 11:34:47 email0 archive: body checksum mismatch: incoming: 1228
UTSL is easier way.
% make patch # make sure patch.20.gz is applied.
% view work.i386/distribute-pl19/distribute.c
and see how bodysum is updated and write_index() is called.
Call the write_index after the sum is calclated (but I'm not sure
the origianl intention of the change in patch.20.gz).
State-Changed-When: Thu Sep 26 01:17:01 PDT 2002
Wouldn't the original authors be a better place to report this,
especially if you're unclear about parts of the source?
State-Changed-When: Fri Jan 10 13:57:36 PST 2003
Responsible-Changed-When: Fri Jan 10 13:57:36 PST 2003
Feel free to commit a fix.
From: David Holland <email@example.com>
Subject: Re: pkg/18422: mail/distribute: write_index() is called too early.
Date: Mon, 10 Aug 2015 06:34:48 +0000
Looking at this, it seems that patch20, or at least that part of it,
is just wrong.
Is there an upstream for this package? The pkgsrc package doesn't
appear to have been touched in any substantive way since it was first
added in 1999.
If there is no upstream, is the package worth maintaining? It is not
very large and it should be fairly straightforward to wade into it and
figure out what that change was supposed to do, and fix it properly;
but it's only worth doing if anyone's going to actually use the
David A. Holland
$NetBSD: query-full-pr,v 1.39 2013/11/01 18:47:49 spz Exp $
$NetBSD: gnats_config.sh,v 1.8 2006/05/07 09:23:38 tsutsui Exp $
Copyright © 1994-2007
The NetBSD Foundation, Inc. ALL RIGHTS RESERVED.