Am 04.03.21 um 21:17 schrieb Jeff Hostetler via GitGitGadget:
quoted hunk ↗ jump to hunk
From: Jeff Hostetler <redacted>
When checking to see if our unix domain socket was stolen, also check
whether the st.st_dev and st.st_mode fields changed (in addition to
the st.st_ino field).
The inode by itself is not unique; it is only unique on a given
device.
Signed-off-by: Jeff Hostetler <redacted>
---
unix-stream-server.c | 5 ++++-
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/unix-stream-server.c b/unix-stream-server.c
index f00298ca7ec3..366ece69306b 100644
--- a/unix-stream-server.c
+++ b/unix-stream-server.c
@@ -120,8 +120,11 @@ int unix_stream_server__was_stolen(
if (st_now.st_ino != server_socket->st_socket.st_ino)
return 1;
+ if (st_now.st_dev != server_socket->st_socket.st_dev)
+ return 1;
- /* We might also consider the ctime on some platforms. */
Why remove that comment? (This change is not mentioned in the commit
message.)
quoted hunk ↗ jump to hunk
+ if (!S_ISSOCK(st_now.st_mode))
+ return 1;
return 0;
}