vultr/openclaw
1
0
Fork 0
mirror of https://github.com/openclaw/openclaw.git synced 2026-06-02 19:15:16 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix: parse qa parent pid strictly

Browse Source
This commit is contained in:
Peter Steinberger
2026-05-28 14:40:58 -04:00
parent 528371e7a4
commit b2fdbc53e8
2 changed files with 5 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
src/cli/gateway-cli/qa-parent-watchdog.test.ts
View File

@@ -13,6 +13,10 @@ describe("installQaParentWatchdog", () => {
expect(
installQaParentWatchdog({ env: { [QA_PARENT_PID_ENV]: "not-a-pid" }, ownPid: 10 }),
).toBeNull();
expect(
installQaParentWatchdog({ env: { [QA_PARENT_PID_ENV]: "0x10" }, ownPid: 10 }),
).toBeNull();
expect(installQaParentWatchdog({ env: { [QA_PARENT_PID_ENV]: "1e3" }, ownPid: 10 })).toBeNull();
});
it("exits when the QA parent process disappears", async () => {

2
src/cli/gateway-cli/qa-parent-watchdog.ts
View File

@@ -40,7 +40,7 @@ function resolveQaParentPid(env: NodeJS.ProcessEnv, ownPid: number): number | nu
if (!raw) {
return null;
}
const parentPid = Number(raw);
const parentPid = /^\d+$/.test(raw) ? Number(raw) : Number.NaN;
if (!Number.isSafeInteger(parentPid) || parentPid <= 0 || parentPid === ownPid) {
return null;
}