1641611 - Repeatable assertion on PermissionManager::UpdateDB() failure in Execute()

Status: RESOLVED FIXED

(Core :: Permission Manager, defect, P2)

Description

[Randell Jesup [:jesup] (needinfo me)]

In a debug build a number of URLs will reliably crash after stmt->Update() on the assertion that it succeeds.

https://time.com/5840208/brazil-coronavirus/?utm_source=pocket-newtab
https://getpocket.com/explore/item/alien-at-40-ridley-scott-explains-why-you-don-t-show-the-monster-too-many-times?utm_source=pocket-newtab

Comment 1

[BugBot [:suhaib / :marco/ :calixte]]

The severity field is not set for this bug.
:johannh, could you have a look please?

For more information, please visit auto_nag documentation.

Comment 2

[Johann Hofmann [:johannh]]

Baku, maybe from bug 1363541?

Comment 3

[Morgan Reschenberg [:morgan]]

I've run into this too :) Mine crashes loading twitter.com

Comment 4

[Andrea Marchesini [:baku]]

Can you submit a crash report please?

Comment 5

[Morgan Reschenberg [:morgan]]

(In reply to Andrea Marchesini [:baku] from comment #4)

Can you submit a crash report please?

Filed as a dependency here! https://bugzilla.mozilla.org/show_bug.cgi?id=1646444

Comment 7

[Andrea Marchesini [:baku]]

Randell, are you still able to reproduce this crash? I wonder if you can tell me more about the query execution failure.

Comment 8

[Randell Jesup [:jesup] (needinfo me)]

I'm not able to repro anymore.

Comment 9

[Peter Van der Beken [:peterv]]

I have this in a rr recording, I'll try to get it into pernosco.

Comment 10

[Peter Van der Beken [:peterv]]

This was from twitter.com: https://pernos.co/debug/nPBatIBhFw9hvYxx6FIDmg/index.html

Comment 11

[Andrea Marchesini [:baku]]

Peter, too late! The pernosco session is expired. Are you able to provide a new one?

Comment 12

[Peter Van der Beken [:peterv]]

I still have the recording, I'll try to get it into pernosco again.

Comment 13

[Peter Van der Beken [:peterv]]

https://pernos.co/debug/PQFK1QizS_MP8gLSMoplpA/index.html

Comment 14

[Andrea Marchesini [:baku]]

Attachment: Bug 1641611 - PermissionManager should take the largest ID considering the expired permissions too, r?dimi

Comment 15

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/4454c09afd76
PermissionManager should take the largest ID considering the expired permissions too, r=timhuang

Comment 16

[Sandor Molnar[:smolnar]]

https://hg.mozilla.org/mozilla-central/rev/4454c09afd76