aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorUlrich Drepper <drepper@redhat.com>2005-01-06 21:01:25 +0000
committerUlrich Drepper <drepper@redhat.com>2005-01-06 21:01:25 +0000
commit5cf53cc2083e0bc77d6d7301fff8c31038a410a2 (patch)
tree1d59ed6fc2eb95bfd1946b79e5b4761978afe5b4
parentbb803bff5cb97b3de94896aba1c4ec0d67227524 (diff)
downloadglibc-5cf53cc2083e0bc77d6d7301fff8c31038a410a2.zip
glibc-5cf53cc2083e0bc77d6d7301fff8c31038a410a2.tar.gz
glibc-5cf53cc2083e0bc77d6d7301fff8c31038a410a2.tar.bz2
Update.
2005-01-06 Ulrich Drepper <drepper@redhat.com> * misc/sys/cdefs.h: Define __wur. * libio/stdio.h: Use __wur for a number of interfaces. * posix/unistd.h: Likewise. * posix/regex_internal.c (free_state): Free word_trtable.
-rw-r--r--ChangeLog7
-rw-r--r--WUR-REPORT25
-rw-r--r--misc/sys/cdefs.h8
-rw-r--r--posix/regex_internal.c1
4 files changed, 41 insertions, 0 deletions
diff --git a/ChangeLog b/ChangeLog
index e251cc7..5c62032 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,9 @@
+2005-01-06 Ulrich Drepper <drepper@redhat.com>
+
+ * misc/sys/cdefs.h: Define __wur.
+ * libio/stdio.h: Use __wur for a number of interfaces.
+ * posix/unistd.h: Likewise.
+
2004-12-29 Jakub Jelinek <jakub@redhat.com>
* sysdeps/ia64/fpu/libm_support.h (__libm_error_support): Use
@@ -342,6 +348,7 @@
(transit_state): Expect state->trtable to be a 256-item
transition table. Reorganize code to have less tests in
the common case, and to save an indentation level.
+ * posix/regex_internal.c (free_state): Free word_trtable.
2004-12-21 Jakub Jelinek <jakub@redhat.com>
diff --git a/WUR-REPORT b/WUR-REPORT
new file mode 100644
index 0000000..a8b2860
--- /dev/null
+++ b/WUR-REPORT
@@ -0,0 +1,25 @@
+<unistd.h>:
+
+lssek: Probably should be __wur but lseek(fd,SEEK_SET,0) will succeed if
+ the descriptor is fine.
+lseek64: same
+
+setuid: will always succeed given correct privileges, so there might
+ be places which don't check for it.
+setreuid: same
+seteuid: same
+setgid: same
+setregid: same
+setegid: same
+setresuid: same
+setresgid: same
+
+
+<stdio.h>:
+
+setvbuf: if stream and buffer are fine and other parameters constant,
+ it cannot really fail.
+fseek: see lseek
+fseeko: likewise
+fgetpos: similarly
+fsetpos: likewise
diff --git a/misc/sys/cdefs.h b/misc/sys/cdefs.h
index 475cf62..6ac2101 100644
--- a/misc/sys/cdefs.h
+++ b/misc/sys/cdefs.h
@@ -255,6 +255,14 @@
# define __nonnull(params)
#endif
+/* If fortification mode, we warn about unused results of certain
+ function calls which can lead to problems. */
+#if __GNUC_PREREQ (3,4) && __USE_FORTIFY_LEVEL > 0
+# define __wur __attribute__ ((__warn_unused_result__))
+#else
+# define __wur /* Ignore */
+#endif
+
/* It is possible to compile containing GCC extensions even if GCC is
run in pedantic mode if the uses are carefully marked using the
`__extension__' keyword. But this is not generally available before
diff --git a/posix/regex_internal.c b/posix/regex_internal.c
index 001b50b..f15cb57 100644
--- a/posix/regex_internal.c
+++ b/posix/regex_internal.c
@@ -1668,6 +1668,7 @@ free_state (state)
re_free (state->entrance_nodes);
}
re_node_set_free (&state->nodes);
+ re_free (state->word_trtable);
re_free (state->trtable);
re_free (state);
}