Opened 2 years ago

Closed 2 years ago

#17960 closed bug (fixed)

Webpositive (master) - crash on quit after viewing Google image

Reported by: vidrep Owned by: pulkomandy
Priority: normal Milestone: R1/beta4
Component: Applications/WebPositive Version: R1/beta3
Keywords: Cc:
Blocked By: Blocking:
Platform: All

Description

hrev56490 x86_64

HaikuWebKit 1.8.4 WebKit 615.1.4

After viewing a Google image WebPositive crashes on quit.

Debug report attached

Attachments (1)

WebPositive-1273-debug-28-09-2022-20-55-06.report (26.3 KB ) - added by vidrep 2 years ago.

Download all attachments as: .zip

Change History (8)

comment:1 by madmax, 2 years ago

I can't seem to get it with WebPositive, but I do almost always with HaikuLauncher. In my case the leaked references are for the static context in platformSession.

comment:3 by pulkomandy, 2 years ago

I am building the 1.9.0 release with curl instead of netservices API, so that code is not used. That should be ok for the beta 4 version.

How did you identify which allocation was problematic? I could not find it.

comment:4 by pulkomandy, 2 years ago

Also isn't there already a default URL context in the services kit? Shouldn't we use that one instead?

comment:5 by madmax, 2 years ago

How did you identify which allocation was problematic? I could not find it.

No analysis, so I don't even know where the references are aquired and never released. I get easily confused with simple pointers and references, so if you add BReferences to the mix...

So brute force. I got obsessive, added a field to BReferenceable (and BReference, for good measure), filled it in every constructor and added it to the log. That showed it to be a BUrlContext, and then I did something similar to find out which instantiation it was.

Also isn't there already a default URL context in the services kit?

There seems to be one in UrlRequest.

comment:6 by madmax, 2 years ago

This one should be fixed by the change to curl, or fe216a33 if your go back to netservices. I have not seen the crash again, but I only got it with HaikuLauncher, not with WebPositive as in the report.

comment:7 by pulkomandy, 2 years ago

Milestone: UnscheduledR1/beta4
Resolution: fixed
Status: newclosed
Note: See TracTickets for help on using tickets.