Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 25 Jun 2001 00:10:20 +0300
From:      Valentin Nechayev <netch@iv.nn.kiev.ua>
To:        Stephen Montgomery-Smith <stephen@math.missouri.edu>, ru@freebsd.org
Cc:        freebsd-stable@freebsd.org, current@freebsd.org
Subject:   Re: Problems with md5 -p
Message-ID:  <20010625001020.B4663@iv.nn.kiev.ua>
In-Reply-To: <3B362E67.5BFC9AF2@math.missouri.edu>; from stephen@math.missouri.edu on Sun, Jun 24, 2001 at 01:16:07PM -0500
References:  <3B35F852.2290E18F@math.missouri.edu> <20010624194339.A717@iv.nn.kiev.ua> <3B362E67.5BFC9AF2@math.missouri.edu>

next in thread | previous in thread | raw e-mail | index | archive | help
 Sun, Jun 24, 2001 at 13:16:07, stephen (Stephen Montgomery-Smith) wrote about "Re: Problems with md5 -p": 

> OK, I'm going to make this into a PR so that it gets fixed soon.  (The
> problem in stable appeared between May 19 and June 16.)

Yes, it appeared with commits:

ru          2001/05/22 03:33:44 PDT

  Modified files:
    sbin/md5             Makefile md5.c
  Removed files:
    sbin/md5             global.h
  Log:
  Fix argument processing.
  Make this compile with WARNS=2.

  PR:           bin/27524
  MFC after:    3 days

  Revision  Changes    Path
  1.5       +3 -1      src/sbin/md5/Makefile
  1.22      +46 -52    src/sbin/md5/md5.c



ru          2001/05/26 05:08:35 PDT

  Modified files:        (Branch: RELENG_4)
    sbin/md5             md5.c
  Removed files:         (Branch: RELENG_4)
    sbin/md5             global.h
  Log:
  MFC: fix argument processing.

  Revision  Changes    Path
  1.20.2.2  +46 -52    src/sbin/md5/md5.c


Before them the case when MDFilter(0) should be called, checked separately
(argc==1). After them it is not checked, "fix argument processing"
is somehow wrong. IMO these commits should be reverted.


> Valentin Nechayev wrote:
> > 
> >  Sun, Jun 24, 2001 at 09:25:22, stephen (Stephen Montgomery-Smith) wrote about "Problems with md5 -p":
> > 
> > I reproduce it stably on my -current. The second checksum is constant
> > and it is MD5 checksum of an empty stream:
> > 
> > root@iv:/usr/HEAD/src/sbin/md5##md5 </dev/null
> > d41d8cd98f00b204e9800998ecf8427e
> > 
> > A fix:
> > 
> > --- md5.c.orig  Mon Jun  4 00:38:02 2001
> > +++ md5.c       Sun Jun 24 19:37:13 2001
> > @@ -65,7 +65,7 @@
> >                 switch (ch) {
> >                 case 'p':
> >                         MDFilter(1);
> > -                       break;
> > +                       exit(0);
> >                 case 'q':
> >                         qflag = 1;
> >                         break;
> > 
> > This avoids determination of other options, but this does not conflict
> > directly with man page.
> > 
> > Moreover such exit(0) should be applied not only with -p, but also with
> > -x, -t and -s: all these options should not gather any input files.
> > Patch is trivial.
> > 
> > > Suppose I have a file xxx.  If I type
> > >
> > > md5 -p < xxx
> > >
> > > it should return the contents of the file followed by its md5 number:
> > >
> > > Some junk in the file
> > >
> > > 334911f8bcde69fe8edac561197e876f
> > >
> > > But now I get two numbers:
> > >
> > > Some junk in the file
> > >
> > > 334911f8bcde69fe8edac561197e876f
> > > d41d8cd98f00b204e9800998ecf8427e
> > >
> > > This is using FreeBSD stable of June 16.  (Maybe this has been fixed
> > 
> > > more recently - please tell me of it has.  It is a bit tricky for me to
> > > update sources because I use CTM which has been out recently - probably
> > > for just this very reason.  But if I know the problem has been fixed
> > > then I will go through the effort of using cvsup.)


/netch

To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe freebsd-current" in the body of the message




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20010625001020.B4663>