This is an automated email from Gerrit.

Tim Newsome ([email protected]) just uploaded a new patch set to Gerrit, which 
you can find at http://openocd.zylin.com/4316

-- gerrit

commit 12b6f6f8972165d76094552a8ac2ef9a33777144
Author: Tim Newsome <[email protected]>
Date:   Tue Dec 26 13:44:26 2017 -0800

    Support listening on port 0.
    
    When listening on port 0, the system will assign a random open port. We
    use this to run multiple OpenOCD instances against multiple simulators
    as part of regression testing. This mechanism means the various test
    instances don't have to coordinate to ensure they don't reuse any ports.
    
    The required changes are minimal:
    1. Don't increment the port number when it's 0.
    2. Print out which port was assigned by the system.
    
    Change-Id: I404c801fc405e9d8eb8420562c02e78d4db6242f
    Signed-off-by: Tim Newsome <[email protected]>

diff --git a/src/server/gdb_server.c b/src/server/gdb_server.c
index 626179c..c48a812 100644
--- a/src/server/gdb_server.c
+++ b/src/server/gdb_server.c
@@ -3088,7 +3088,13 @@ static int gdb_target_add_one(struct target *target)
                if (!*end) {
                        if (parse_long(gdb_port_next, &portnumber) == ERROR_OK) 
{
                                free(gdb_port_next);
-                               gdb_port_next = alloc_printf("%d", 
portnumber+1);
+                               if (portnumber) {
+                                       gdb_port_next = alloc_printf("%d", 
portnumber+1);
+                               } else {
+                                       /* Don't increment if gdb_port is 0, 
since we're just
+                                        * trying to allocate an unused port. */
+                                       gdb_port_next = alloc_printf("0");
+                               }
                        }
                }
        }
diff --git a/src/server/server.c b/src/server/server.c
index 517d62a..3c806f7 100644
--- a/src/server/server.c
+++ b/src/server/server.c
@@ -298,6 +298,12 @@ int add_service(char *name,
                        free_service(c);
                        return ERROR_FAIL;
                }
+
+               struct sockaddr_in addr_in;
+               socklen_t addr_in_size = sizeof(addr_in);
+               getsockname(c->fd, (struct sockaddr *)&addr_in, &addr_in_size);
+               LOG_INFO("Listening on port %d for %s connections",
+                               ntohs(addr_in.sin_port), name);
        } else if (c->type == CONNECTION_STDINOUT) {
                c->fd = fileno(stdin);
 

-- 

------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
OpenOCD-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/openocd-devel

Reply via email to