diff options
| author | Daniel Sabogal <dsabogalcc@gmail.com> | 2016-09-23 21:10:07 -0400 |
|---|---|---|
| committer | Rich Felker <dalias@aerifal.cx> | 2016-09-24 01:10:23 -0400 |
| commit | 8d7a3f40c8de414122eb8eb2131291782ee8ed15 (patch) | |
| tree | d49133e15f34e1690820db11d60d6541e5926053 | |
| parent | d917ef0b8468f77634df59a1a6650aff5adb6b6a (diff) | |
| download | musl-8d7a3f40c8de414122eb8eb2131291782ee8ed15.tar.gz | |
fix getservby*_r result pointer value on error
this is a clone of the fix to the gethostby*_r functions in
commit fe82bb9b921be34370e6b71a1c6f062c20999ae0. the man pages
document that the getservby*_r functions set this pointer to
NULL if there was an error or if no record was found.
| -rw-r--r-- | src/network/getservbyname_r.c | 2 | ||||
| -rw-r--r-- | src/network/getservbyport_r.c | 1 |
2 files changed, 3 insertions, 0 deletions
diff --git a/src/network/getservbyname_r.c b/src/network/getservbyname_r.c index 056c2f33..ad3d6164 100644 --- a/src/network/getservbyname_r.c +++ b/src/network/getservbyname_r.c @@ -15,6 +15,8 @@ int getservbyname_r(const char *name, const char *prots, struct service servs[MAXSERVS]; int cnt, proto, align; + *res = 0; + /* Align buffer */ align = -(uintptr_t)buf & ALIGN-1; if (buflen < 2*sizeof(char *)+align) diff --git a/src/network/getservbyport_r.c b/src/network/getservbyport_r.c index a0a7caec..0ae0e415 100644 --- a/src/network/getservbyport_r.c +++ b/src/network/getservbyport_r.c @@ -20,6 +20,7 @@ int getservbyport_r(int port, const char *prots, if (r) r = getservbyport_r(port, "udp", se, buf, buflen, res); return r; } + *res = 0; /* Align buffer */ i = (uintptr_t)buf & sizeof(char *)-1; |