Opened 15 years ago
Closed 15 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)
Change History (3)
by , 15 years ago
comment:1 by , 15 years ago
| Milestone: | → 1.1.3 |
|---|---|
| Status: | new → assigned |
Looks like the fix in r12208 was wrong.
The patch looks like the right fix to me from a quick look.
comment:2 by , 15 years ago
| Milestone: | 1.1.3 → 1.0.16 |
|---|---|
| Resolution: | → fixed |
| Status: | assigned → closed |
Backported for 1.0.16 in r13444.
Note:
See TracTickets
for help on using tickets.
is this the right way to fix?