]> git.ipfire.org Git - thirdparty/valgrind.git/commitdiff
Make sure socket file is created in socket_close testcase
authorMark Wielaard <mark@klomp.org>
Fri, 15 Mar 2024 11:55:23 +0000 (12:55 +0100)
committerMark Wielaard <mark@klomp.org>
Fri, 15 Mar 2024 11:55:23 +0000 (12:55 +0100)
If the socket file already existed the path name wouldn't be recorded.
Explicitly unlink it before opening and binding the socket.  Adjust
none/tests/socket_close.stderr.exp for new path name and line numbers.

none/tests/socket_close.c
none/tests/socket_close.stderr.exp

index 99d277229fea2c71f7c355aef4810c5cdbbb100e..402c0a5d554b6d408d0252bab9f3e99dec950184 100644 (file)
@@ -6,21 +6,21 @@
 #include <string.h>
 #include <sys/un.h>
 
-
+const char *SPATH = "/tmp/vgtest-foofrob";
 int socket_fd;
 
-void *open_socket()
+void open_socket()
 {
+  unlink (SPATH); /* Make sure socket path doesn't exist yet.  */
+
   socket_fd = socket(AF_UNIX, SOCK_STREAM, 0);
   fprintf (stderr, "Open socket %d\n", socket_fd);
   struct sockaddr_un  my_addr;
 
   memset(&my_addr, 0, sizeof(my_addr));
   my_addr.sun_family = AF_UNIX;
-  strncpy(my_addr.sun_path, "/tmp/foofrob", sizeof(my_addr.sun_path) - 1);
+  strncpy(my_addr.sun_path, SPATH, sizeof(my_addr.sun_path) - 1);
   bind(socket_fd, (struct sockaddr *) &my_addr, sizeof(my_addr));
-
-  return NULL;
 }
 
 int main ()
index bec4d637016d7976282c65dcff520a5e7b19b75c..74b06244d61a1669e69ad22111f763ecd182cb0e 100644 (file)
@@ -1,7 +1,7 @@
 Open socket 3
 close socket_fd 3
 and close the socket again 3
-File descriptor 3: AF_UNIX socket 3: <unknown> is already closed
+File descriptor 3: AF_UNIX socket 3: /tmp/vgtest-foofrob is already closed
    at 0x........: close (in /...libc...)
    by 0x........: main (socket_close.c:37)
  Previously closed
@@ -9,5 +9,5 @@ File descriptor 3: AF_UNIX socket 3: <unknown> is already closed
    by 0x........: main (socket_close.c:33)
  Originally opened
    at 0x........: socket (in /...libc...)
-   by 0x........: open_socket (socket_close.c:14)
+   by 0x........: open_socket (socket_close.c:16)
    by 0x........: main (socket_close.c:28)