Opened 11 years ago

Closed 11 years ago

#398 closed defect (fixed)

unsuccessful lock attempt leaves zombie

Reported by: Dmitry Liakh Owned by: Olly Betts
Priority: normal Milestone: 1.0.16
Component: Backend-Flint Version: 1.0.15
Severity: normal Keywords:
Cc: Blocked By:
Blocking: Operating System: Linux

Description

It seems pid may be passed unitialized to waitpid(2) in Flintlock::lock() (see `pid = child' assignment somewhere around line 188 of backends/flint/flint_lock.cc and a few lines above, particularly `if (why != SUCCESS) break;')

Attachments (1)

patch-aa (330 bytes ) - added by Dmitry Liakh 11 years ago.
is this the right way to fix?

Download all attachments as: .zip

Change History (3)

by Dmitry Liakh, 11 years ago

Attachment: patch-aa added

is this the right way to fix?

comment:1 by Olly Betts, 11 years ago

Milestone: 1.1.3
Status: newassigned

Looks like the fix in r12208 was wrong.

The patch looks like the right fix to me from a quick look.

comment:2 by Olly Betts, 11 years ago

Milestone: 1.1.31.0.16
Resolution: fixed
Status: assignedclosed

Backported for 1.0.16 in r13444.

Note: See TracTickets for help on using tickets.