Closed Bug 470394 Opened 11 years ago Closed 11 years ago

Guard all SQL transactions in case of database corruption

Categories

(Calendar :: Provider: Local Storage, defect)

defect
Not set

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: Fallen, Assigned: Fallen)

Details

Attachments

(1 file)

Attached patch Fix - v1Splinter Review
I've recently had someone with an obviously corrupted database. All his events were not showing due to the fact that getting additional data threw an exception. We were able to restore his data by catching the exception and resetting the statement.

This patch takes care of that for all remaining SQL transactions in the storage provider.
Attachment #353794 - Flags: review?(daniel.boelzle)
Comment on attachment 353794 [details] [diff] [review]
Fix - v1

looks good; r=dbo with s/ERROR/cal.ERROR/.
Attachment #353794 - Flags: review?(daniel.boelzle) → review+
Pushed to comm-central <http://hg.mozilla.org/comm-central/rev/260b5e14bef1>

-> FIXED
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → 1.0
These bugs are likely targeted at Lightning 1.0b1, not Lightning 1.0. If this change was done in error, please adjust the target milestone to its correct value. To filter on this bugspam, you can use "lightning-10-target-move".
Target Milestone: 1.0 → 1.0b1
You need to log in before you can comment on or make changes to this bug.