diff options
| author | Android (Google) Code Review <android-gerrit@google.com> | 2009-05-27 03:29:35 -0700 |
|---|---|---|
| committer | Android (Google) Code Review <android-gerrit@google.com> | 2009-05-27 03:29:35 -0700 |
| commit | cdb68bf8ce26448cfe173d86f1c074e167c3420f (patch) | |
| tree | 1b726243e78842727d932bd197010441589232fc /libc/netbsd/net/getservent.c | |
| parent | 5c106fcaa5b9d0a3ebeb2e8cdca589129d6a7ebd (diff) | |
| parent | 2735b33519595da5d51b79757ccce730021a9228 (diff) | |
| download | bionic-cdb68bf8ce26448cfe173d86f1c074e167c3420f.zip bionic-cdb68bf8ce26448cfe173d86f1c074e167c3420f.tar.gz bionic-cdb68bf8ce26448cfe173d86f1c074e167c3420f.tar.bz2 | |
Merge change 2470 into donut
* changes:
Fix getservent() so that it returns s_port in network byte order. Also add a new document detailing known issues in the C library.
Diffstat (limited to 'libc/netbsd/net/getservent.c')
| -rw-r--r-- | libc/netbsd/net/getservent.c | 9 |
1 files changed, 7 insertions, 2 deletions
diff --git a/libc/netbsd/net/getservent.c b/libc/netbsd/net/getservent.c index 45207df..65fbd7e 100644 --- a/libc/netbsd/net/getservent.c +++ b/libc/netbsd/net/getservent.c @@ -27,6 +27,7 @@ */ #include <sys/cdefs.h> #include <sys/types.h> +#include <sys/endian.h> #include <netdb.h> #include "servent.h" #include "services.h" @@ -54,6 +55,7 @@ getservent_r( res_static rs ) int namelen; int nn,count; int total = 0; + int port; char* p2; p = rs->servent_ptr; @@ -92,9 +94,12 @@ getservent_r( res_static rs ) memcpy( rs->servent.s_name, p+1, namelen ); rs->servent.s_name[namelen] = 0; p += 1 + namelen; - rs->servent.s_port = ((((unsigned char*)p)[0] << 8) | - ((unsigned char*)p)[1]); + /* s_port must be in network byte order */ + port = ((((unsigned char*)p)[0] << 8) | + ((unsigned char*)p)[1]); + + rs->servent.s_port = htons(port); rs->servent.s_proto = p[2] == 't' ? "tcp" : "udp"; p += 4; /* skip port(2) + proto(1) + aliascount(1) */ |