Opened 2 years ago

Closed 2 years ago

#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
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 2 years ago.

Download all attachments as: .zip

Change History (8)

by axeld, 2 years ago

Attachment: advisory-locking-bug.png added

comment:1 by axeld, 2 years ago

Description: modified (diff)

comment:2 by pulkomandy, 2 years ago

Blocking: 14118 added

comment:3 by phoudoin, 2 years ago

Blocking: 14120 added

comment:4 by korli, 2 years ago

Blocking: 14119 added

comment:5 by waddlesplash, 2 years ago

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 by waddlesplash, 2 years ago

Possibly fixed by hrev51970.

comment:7 by pulkomandy, 2 years ago

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.