about summary refs log tree commit diff
diff options
context:
space:
mode:
authorMike Frysinger <vapier@gentoo.org>2015-07-27 23:43:09 -0400
committerMike Frysinger <vapier@gentoo.org>2015-07-27 23:45:49 -0400
commit1eb8bf804937bea53f5e8cfa2f84d256373cdb64 (patch)
treec2a8b2c7aa8dd6d331832589fcf3787b90d1f61d
parent3df5cd98371dd9f10b6853efa4d54d359614007d (diff)
downloadglibc-1eb8bf804937bea53f5e8cfa2f84d256373cdb64.tar.gz
glibc-1eb8bf804937bea53f5e8cfa2f84d256373cdb64.tar.xz
glibc-1eb8bf804937bea53f5e8cfa2f84d256373cdb64.zip
pwd.h: revert __nonnull markings on putpwent [BZ #18641]
This function actually checks for NULL arguments and the API has been
tenatively documented as using EINVAL in that case.  We can debate
leaving it this way, but it should be done after the pending release.
-rw-r--r--ChangeLog5
-rw-r--r--pwd/pwd.h2
2 files changed, 6 insertions, 1 deletions
diff --git a/ChangeLog b/ChangeLog
index 0498847a52..cf1efcce19 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,8 @@
+2015-07-27  Mike Frysinger  <vapier@gentoo.org>
+
+	[BZ #18641]
+	* pwd/pwd.h (putpwent): Delete __nonnull markings.
+
 2015-07-27  Andreas Schwab  <schwab@suse.de>
 
 	* conform/conformtest.pl (checknamespace): Filter out string
diff --git a/pwd/pwd.h b/pwd/pwd.h
index fcfb2ab0e7..70a051deb1 100644
--- a/pwd/pwd.h
+++ b/pwd/pwd.h
@@ -100,7 +100,7 @@ extern struct passwd *fgetpwent (FILE *__stream) __nonnull ((1));
    or due to the implementation it is a cancellation point and
    therefore not marked with __THROW.  */
 extern int putpwent (const struct passwd *__restrict __p,
-		     FILE *__restrict __f) __nonnull ((1, 2));
+		     FILE *__restrict __f);
 #endif
 
 /* Search for an entry with a matching user ID.