apr-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From traw...@apache.org
Subject svn commit: r1551666 - /apr/apr/branches/1.5.x/test/testsock.c
Date Tue, 17 Dec 2013 19:45:49 GMT
Author: trawick
Date: Tue Dec 17 19:45:49 2013
New Revision: 1551666

URL: http://svn.apache.org/r1551666
Log:
Follow-up to r1004521:

Check the return code like in trunk r1089424.  As it is,
it normally bypasses the testcase and leaves the bound socket
open.

Modified:
    apr/apr/branches/1.5.x/test/testsock.c

Modified: apr/apr/branches/1.5.x/test/testsock.c
URL: http://svn.apache.org/viewvc/apr/apr/branches/1.5.x/test/testsock.c?rev=1551666&r1=1551665&r2=1551666&view=diff
==============================================================================
--- apr/apr/branches/1.5.x/test/testsock.c (original)
+++ apr/apr/branches/1.5.x/test/testsock.c Tue Dec 17 19:45:49 2013
@@ -350,8 +350,8 @@ static void test_get_addr(abts_case *tc,
 
     APR_ASSERT_SUCCESS(tc, "create subpool", apr_pool_create(&subp, p));
 
-    if ((ld = setup_socket(tc)) != APR_SUCCESS)
-        return;
+    ld = setup_socket(tc);
+    if (!ld) return;
 
     APR_ASSERT_SUCCESS(tc,
                        "get local address of bound socket",



Mime
View raw message