Received: (at submit) by bugs.debian.org; 19 Nov 1996 00:41:54 +0000
Received: (qmail 17317 invoked from network); 19 Nov 1996 00:41:53 -0000
Received: from cartridge.i-connect.net (qmailr@206.139.73.12)
  by master.debian.org with SMTP; 19 Nov 1996 00:41:53 -0000
Received: (qmail 25126 invoked from network); 19 Nov 1996 00:27:53 -0000
Received: from waldo-1.npr.legent.com (HELO lachman.com) (192.35.57.109)
  by cartridge.i-connect.net with SMTP; 19 Nov 1996 00:27:52 -0000
Received: by lachman.com
	id m0vPcwG-001GNbC
	(Debian /\oo/\ Smail3.1.29.1 #29.37); Mon, 18 Nov 96 17:17 CST
Message-Id: <m0vPcwG-001GNbC@lachman.com>
Date: Mon, 18 Nov 96 17:17 CST
From: Brad Bosch <brad@lachman.com>
To: submit@bugs.debian.org
Subject: id-utils version 3.2 fix for idfile.c
Return-Receipt-To: brad
X-Debian-CC: bug-gnu-utils@prep.ai.mit.edu, branderh@debian.org, gkm@gnu.ai.mit.edu
Mime-Version: 1.0 (generated by tm-edit 7.93)
Content-Type: text/plain; charset=US-ASCII

Package: id-utils
Version: 3.2-1

One more try on reporting this...

I tried to report this a while back but got no response from anyone I
sent it to and I can find no Debian bug for it so I will try again.
If the involved parties would please acknowledge receipt of this I
would appreciate it.

I found and fixed a trivial bug in locate_id_file_name() which I
discovered while running gid (from emacs) in a child of a child of the
directory where the ID file was stored.  The ID file was not found.

This bug also exists in at least the Debian version of release 3.1.  I
have included a diff and I am CCing the GNU folks (Thanks to Greg for
providing this software!).  I hope this is the right way to report
this to both groups.

--Brad Bosch
brad@lachman.com

--- old/idfile.c	Mon Jul  8 23:19:08 1996
+++ ./idfile.c	Tue Aug 27 11:38:09 1996
@@ -85,7 +85,7 @@
       if (stat (file_name_buffer, &statb) != 0)
 	return NULL;
     }
-  while (!((statb.st_ino == rootb.st_ino) ||
+  while (!((statb.st_ino == rootb.st_ino) &&
 	   (statb.st_dev == rootb.st_dev)));
   return NULL;
 }