#14121 closed bug (fixed)

Advisory locking leaked

Reported by: axeld Owned by: nobody
Priority: blocker Milestone: R1/beta1
Component: System/Kernel Version: R1/Development
Keywords: Cc:
Blocked By: Blocking: #14118, #14119, #14120
Has a Patch: no Platform: All

Description (last modified by axeld)

The assertion added in hrev51893 is hit when you actually use advisory locking (like, for example, smbd does).

Attachments (1)

advisory-locking-bug.png (36.8 KB) - added by axeld 15 months ago.

Download all attachments as: .zip

Change History (8)

Changed 15 months ago by axeld

Attachment: advisory-locking-bug.png added

comment:1 Changed 15 months ago by axeld

Description: modified (diff)

comment:2 Changed 15 months ago by pulkomandy

Blocking: 14118 added

comment:3 Changed 15 months ago by phoudoin

Blocking: 14120 added

comment:4 Changed 15 months ago by korli

Blocking: 14119 added

comment:5 Changed 14 months ago by waddlesplash

Priority: criticalblocker
12:57 PM <•PulkoMandy> mh, I tried to grep for it and hit that assert in vnode advisory locking :(
...
12:58 PM <•PulkoMandy> no, easy to trigger on bfs
12:58 PM <•PulkoMandy> (couldn't run the webkit testsuite because of it lately)

comment:6 Changed 14 months ago by waddlesplash

Possibly fixed by hrev51970.

comment:7 Changed 14 months ago by pulkomandy

Resolution: fixed
Status: newclosed

I could build webkit and haiku without hitting the problem, so I guess we're fine now.

Note: See TracTickets for help on using tickets.