Note byte-order on result for parse_addr_port()

svn:r2444

2 files changed, 6 insertions, 6 deletions

diff --git a/src/common/util.c b/src/common/util.c
index 4391ac2411..5065e97216 100644
--- a/src/common/util.c
+++ b/src/common/util.c
@@ -2032,9 +2032,9 @@ int tor_lookup_hostname(const char *name, uint32_t *addr)
  * <b>address</b> is provided, set *<b>address</b> to a copy of the
  * host portion of the string.  If <b>addr</b> is provided, try to
  * resolve the host portion of the string and store it into
- * *<b>addr</b>.  If <b>port</b> is provided, store the port number
- * into *<b>port</b>, or 0 if no port is given.  Return 0 on success,
- * -1 on failure.
+ * *<b>addr</b> (in network byte order).  If <b>port</b> is provided,
+ * store the port number into *<b>port</b>, or 0 if no port is given.
+ * Return 0 on success, -1 on failure.
  */
 int
 parse_addr_port(const char *addrport, char **address, uint32_t *addr,
diff --git a/src/or/test.c b/src/or/test.c
index 4d74651edc..6a13a9db6c 100644
--- a/src/or/test.c
+++ b/src/or/test.c
@@ -640,12 +640,12 @@ test_util() {
   cp = NULL; u32 = 3; u16 = 3;
   test_assert(!parse_addr_port("1.2.3.4", &cp, &u32, &u16));
   test_streq(cp, "1.2.3.4");
-  test_eq(u32, 0x01020304u);
+  test_eq(u32, htonl(0x01020304u));
   test_eq(u16, 0);
   tor_free(cp);
   test_assert(!parse_addr_port("4.3.2.1:99", &cp, &u32, &u16));
   test_streq(cp, "4.3.2.1");
-  test_eq(u32, 0x04030201u);
+  test_eq(u32, htonl(0x04030201u));
   test_eq(u16, 99);
   tor_free(cp);
   test_assert(!parse_addr_port("nonexistent.address:4040", &cp, NULL, &u16));
@@ -654,7 +654,7 @@ test_util() {
   tor_free(cp);
   test_assert(!parse_addr_port("localhost:9999", &cp, &u32, &u16));
   test_streq(cp, "localhost");
-  test_eq(u32, 0x7f000001u);
+  test_eq(u32, htonl(0x7f000001u));
   test_eq(u16, 9999);
   tor_free(cp);
   u32 = 3;