Opened 7 years ago

Closed 7 years ago

#13211 closed bug (fixed)

kernel panic when using socketsetopt

Reported by: raefaldhia Owned by: axeld
Priority: normal Milestone: R1
Component: System/Kernel Version: R1/Development
Keywords: gsoc2017 Cc:
Blocked By: Blocking:
Platform: All

Description (last modified by diver)

This issue doesn't always appear, but in some situations it appears. ​https://github.com/haikuports/haikuports/pull/1076#issuecomment-272686504

kernel panic

Attachments (3)

Screenshot from 2017-01-15 17-24-32.png​ (107.6 KB ) - added by raefaldhia 7 years ago.
kernel panic
0001-fixed-crossing-link-fields-from-different-hash-table.patch​ (3.3 KB ) - added by Carzil 7 years ago.
0001-fixed-crossing-link-fields-from-different-hash-table.2.patch​ (3.2 KB ) - added by Carzil 7 years ago.

Download all attachments as: .zip

Change History (13)

by raefaldhia, 7 years ago

Attachment: Screenshot from 2017-01-15 17-24-32.png​ added

kernel panic

comment:1 by diver, 7 years ago

Component: - General → System/Kernel
Description: modified (diff)
Milestone: Unscheduled → R1
Owner: changed from nobody to axeld

by Carzil, 7 years ago

Attachment: 0001-fixed-crossing-link-fields-from-different-hash-table.patch​ added

comment:2 by Carzil, 7 years ago

patch: 0 → 1

comment:3 by Carzil, 7 years ago

Hello! I've fixed this bug. MulticastGroupInterface (both ipv4 and ipv6) has link field for hash table, but has been inserted in two different tables: in MulticastFilter and sMulticastState. So when ~MulticastFilter was called, it was trying to free group interfaces from other tables.

comment:4 by korli, 7 years ago

The "Global" naming looks weird. Otherwise OK.

comment:5 by waddlesplash, 7 years ago

What else should it be called?

comment:6 by Carzil, 7 years ago

Hmm. fLinkForGlobalMulticastState should be better? Or too long?

comment:7 by korli, 7 years ago

Do fMulticastGroupsLink and MulticastGroupsHashLink() fit?

in reply to:  7 comment:8 by Carzil, 7 years ago

Replying to korli:

Do fMulticastGroupsLink and MulticastGroupsHashLink() fit?

Yes. That's good.

by Carzil, 7 years ago

Attachment: 0001-fixed-crossing-link-fields-from-different-hash-table.2.patch​ added

comment:9 by axeld, 7 years ago

Status: new → in-progress

comment:10 by axeld, 7 years ago

Keywords: gsoc2017 added
Resolution: → fixed
Status: in-progress → closed

Applied in hrev51033, thanks!

Note: See TracTickets for help on using tickets.