diff options
| author | Frédéric Bérat <fberat@redhat.com> | 2025-03-07 18:16:30 +0100 |
|---|---|---|
| committer | Frédéric Bérat <fberat@redhat.com> | 2025-03-21 09:35:42 +0100 |
| commit | 090dfa40a5e46f7c0e4d6e8369bcbbd51267625f (patch) | |
| tree | 472b5b89202d8c7995dc78b9777638ba28cd6fef /debug/tst-fortify.c | |
| parent | 3cdb99d8bb9d0008b2b297080e61d6c10dd66cc8 (diff) | |
| download | glibc-090dfa40a5e46f7c0e4d6e8369bcbbd51267625f.tar.xz glibc-090dfa40a5e46f7c0e4d6e8369bcbbd51267625f.zip | |
Add _FORTIFY_SOURCE support for inet_ntop
- Create the __inet_ntop_chk routine that verifies that the builtin size
of the destination buffer is at least as big as the size given by the
user.
- Redirect calls from inet_ntop to __inet_ntop_chk or __inet_ntop_warn
- Update the abilist for this new routine
- Update the manual to mention the new fortification
Reviewed-by: Florian Weimer <fweimer@redhat.com>
Diffstat (limited to 'debug/tst-fortify.c')
| -rw-r--r-- | debug/tst-fortify.c | 21 |
1 files changed, 21 insertions, 0 deletions
diff --git a/debug/tst-fortify.c b/debug/tst-fortify.c index f8ccc2dff5..cd649369d9 100644 --- a/debug/tst-fortify.c +++ b/debug/tst-fortify.c @@ -23,6 +23,7 @@ #include <assert.h> #include <fcntl.h> +#include <arpa/inet.h> #include <limits.h> #include <locale.h> #include <obstack.h> @@ -1832,6 +1833,26 @@ do_test (void) # endif #endif + struct in6_addr addr6 = {}; + struct in_addr addr = {}; + char addrstr6[INET6_ADDRSTRLEN]; + char addrstr[INET_ADDRSTRLEN]; + + if (inet_ntop (AF_INET6, &addr6, addrstr6, sizeof (addrstr6)) == NULL) + FAIL (); + if (inet_ntop (AF_INET, &addr, addrstr, sizeof (addrstr)) == NULL) + FAIL (); + +#if __USE_FORTIFY_LEVEL >= 1 + CHK_FAIL_START + inet_ntop (AF_INET6, &addr6, buf, INET6_ADDRSTRLEN); + CHK_FAIL_END + + CHK_FAIL_START + inet_ntop (AF_INET, &addr, buf, INET_ADDRSTRLEN); + CHK_FAIL_END +#endif + return ret; } |