When disk full revelation kills password db.

Bug #338138 reported by Antono Vasiljev
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
revelation (Debian)
Fix Released
Unknown
revelation (Ubuntu)
Fix Released
Medium
Stephan Rügamer

Bug Description

Binary package hint: revelation

Steps to reproduce:

1. Open revelation with password file.
2. Make your disk full.
3. Add some passwords.
4. Try to save file.

Expected (normal) result: passwords a savead.
Actually i got empty password.db and message like (unable to save blah blah).

So ALL MY PASSWORDS WAS LOST. (ofcourse i have backup ;))

Revelation should check availible space first and then try to write passwords.db.

It's very serious bug it causes data loss.

Related branches

Changed in revelation (Debian):
status: Unknown → Fix Committed
Revision history for this message
Hew (hew) wrote :

Fixed upstream with revelation 0.4.11-5

Changed in revelation (Ubuntu):
importance: Undecided → Medium
status: New → Fix Committed
Changed in revelation (Debian):
status: Fix Committed → Fix Released
Revision history for this message
ghostonline (ghostonline-deactivatedaccount) wrote :

I have migrated the upstream patch to the latest ubuntu version.

Revision history for this message
Hew (hew) wrote :

Is there a reason we can't just fix it with a merge?

Revision history for this message
ghostonline (ghostonline-deactivatedaccount) wrote :

I had trouble doing the merge (see #520281), so I just tackled the individual problem as a temporary solution. Certainly a merge would be preferred.

Revision history for this message
Stephan Rügamer (sruegamer) wrote :

working on it

Changed in revelation (Ubuntu):
assignee: nobody → Stephan Hermann (shermann)
status: Fix Committed → In Progress
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package revelation - 0.4.11-5ubuntu1

---------------
revelation (0.4.11-5ubuntu1) lucid; urgency=low

  * Merge from debian testing. Remaining changes: (LP: #520281)
    - debian/rules:
      + Fix FTBFS with python >= 2.6
      + Include dh_icons
    - debian/patches/091_panel_category.dpatch:
      + Change GNOME panel category from "Utility" to the correct "Utilities".
    - debian/patches/092_typo_de_translation.dpatch:
      + Fix German translation typo
    - debian/patches/00list:
      + Include 040_xhtml.dpatch
  * Rename Ubuntu specific patches to avoid confusion with Debian's patches:
    - 91_panel_category.dpatch -> 091_panel_category.dpatch
    - 92_typo_de_translation.dpatch -> 092_typo_de_translation.dpatch
  * debian/patches/00list: Update with new Ubuntu specific patch names
  * Dropped changes now applied upstream:
    - debian/patches/94_url_decoding.dpatch:
      - Patches io.py to unquote file url during the normalisation process
  * Dropped changes no longer required:
    - debian/patches/010_intltool.dpatch:
      + Remove intltool files when cleaning
    - debian/patches/020_server_data.dpatch:
      + Remove GNOME_RevelationApplet.server when cleaning
    - debian/patches/93_python26_transition.dpatch:
      + Patches aclocal.m4 to return the correct value for
        cv_python_pyexecdir
  * debian/patches/100_backup_on_save.dpatch: (LP: #338138)
    Patches io.py to create a backup file before saving and restore
    the backup if saving fails

revelation (0.4.11-5) unstable; urgency=low

  * Added VCS information to debian/control
  * Backup file before save (Closes: #524471)
  * urldecode file names (Closes: #518085)
  * fix outdated-autotools-helper-file lintian warning
  * no longer build for python 2.4
  * fix patch-system-but-no-source-readme warning
  * New Standards-Version: 3.8.3
  * fix dh_desktop-is-deprecated warning
 -- Bart Veldstra <email address hidden> Sat, 16 Jan 2010 06:13:18 +0000

Changed in revelation (Ubuntu):
status: In Progress → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.