Skip to content

Commit

Permalink
vsm: Remove PID 1 expectation
Browse files Browse the repository at this point in the history
It turns out varnishd can be the main process of a Docker container when
another process like varnishlog is 'docker exec'uted in the same PID
namespace.

Spotted by @gquintard.
  • Loading branch information
dridi authored and walid-git committed Nov 7, 2024
1 parent 59156c2 commit 4ef61c0
Showing 1 changed file with 0 additions and 1 deletion.
1 change: 0 additions & 1 deletion lib/libvarnishapi/vsm.c
Original file line number Diff line number Diff line change
Expand Up @@ -494,7 +494,6 @@ vsm_running(struct vsm_set *vs, pid_t pid)

if (pid == 0)
return (0);
assert(pid > 1);

if (kill(pid, 0) == 0) {
vs->couldkill = 1;
Expand Down

0 comments on commit 4ef61c0

Please sign in to comment.