Browse Source

when using strsep you should be aware that the delimiter will be replaced

by a nifty '0', so you cant check for a colon instead and dump some
I am sorry, Dave. I can't do that.
OPENBSD_2_1
provos 28 years ago
parent
commit
b221745579
1 changed files with 3 additions and 3 deletions
  1. +3
    -3
      src/lib/libutil/passwd.c

+ 3
- 3
src/lib/libutil/passwd.c View File

@ -1,4 +1,4 @@
/* $OpenBSD: passwd.c,v 1.6 1997/02/13 05:41:38 deraadt Exp $ */
/* $OpenBSD: passwd.c,v 1.7 1997/02/15 09:47:54 provos Exp $ */
/* /*
* Copyright (c) 1987, 1993, 1994, 1995 * Copyright (c) 1987, 1993, 1994, 1995
* The Regents of the University of California. All rights reserved. * The Regents of the University of California. All rights reserved.
@ -287,7 +287,7 @@ pw_scan(bp, pw, flags)
warnx("root uid should be 0"); warnx("root uid should be 0");
return (0); return (0);
} }
if (*p2 != ':') {
if (*p2 != '\0') {
warnx("illegal uid field"); warnx("illegal uid field");
return (0); return (0);
} }
@ -303,7 +303,7 @@ pw_scan(bp, pw, flags)
if (!(p = strsep(&bp, ":"))) /* gid */ if (!(p = strsep(&bp, ":"))) /* gid */
goto fmt; goto fmt;
id = strtoul(p, &p2, 10); id = strtoul(p, &p2, 10);
if (*p2 != ':') {
if (*p2 != '\0') {
warnx("illegal gid field"); warnx("illegal gid field");
return (0); return (0);
} }


Loading…
Cancel
Save