Closed Bug 750312 Opened 12 years ago Closed 4 years ago

Delete permissions database if reading it fails

Categories

(Core :: Permission Manager, enhancement, P3)

Product:
Core
Component:
Permission Manager
Type:
enhancement

Tracking

()

Status:
RESOLVED DUPLICATE of bug 1363541

People

(Reporter: Yoric, Unassigned)

References

Details

(Whiteboard: [necko-backlog]) 

Extract from bug 702344:

«
Another thing I noticed is that we don't delete the permissions database if reading it in fails. We really should do that. Want to take a stab at it? It involves splitting out InitDB() into two methods; InitDB() and TryInitDB(). If the latter returns an error code, InitDB() deletes the database and calls TryInitDB() again. You'll have to reset the statements and db connection like so: http://mxr.mozilla.org/mozilla-central/source/extensions/cookie/nsPermissionManager.cpp#632 and then delete the database like so: http://mxr.mozilla.org/mozilla-central/source/extensions/cookie/nsPermissionManager.cpp#262
» (Dwitte)
Whiteboard: [necko-backlog]
Bulk change to priority: https://bugzilla.mozilla.org/show_bug.cgi?id=1399258
Priority: -- → P1
Bulk change to priority: https://bugzilla.mozilla.org/show_bug.cgi?id=1399258
Priority: P1 → P3
Component: Networking: Cookies → Permission Manager

I fixed this issue in bug 1363541.

Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → DUPLICATE
You need to log in before you can comment on or make changes to this bug.