FS#11260 - lets keybox import its database from an external text file

Attached to Project: Rockbox
Opened by Gerhard Zintel (Gerbil) - Tuesday, 11 May 2010, 23:16 GMT
Last edited by Nils Wallménius (nls) - Wednesday, 18 August 2010, 09:42 GMT
Task Type Patches
Category Plugins
Status Closed
Assigned To No-one
Operating System All players
Severity Low
Priority Normal
Reported Version Release 3.4
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No


This patch is an improvement to the keybox plugin. It lets you import new data from the file "keybox.txt". The file must reside in the same directory as the "keybox.dat" file. For each new entry, three consecutive lines with Title, User and Password information is read from the file and imported into the database.

This patch avoids the need to use the internal text editor to fill the encrypted data base.
This task depends upon

Closed by  Nils Wallménius (nls)
Wednesday, 18 August 2010, 09:42 GMT
Reason for closing:  Rejected
Additional comments about closing:  I'm sorry but this is a too great security risk and even reisk lulling users into false security.
Comment by Gerhard Zintel (Gerbil) - Tuesday, 11 May 2010, 23:19 GMT
It seems that the attachment of the file has not worked
Comment by Gerhard Zintel (Gerbil) - Wednesday, 12 May 2010, 13:54 GMT
The new diff file contains the above attached patch and additionally an update for the manual in keybox.tex.

Tested in the Sansa e200 player in the simulator.
Comment by Gerhard Zintel (Gerbil) - Thursday, 13 May 2010, 11:08 GMT
Reworked version of patch with:

* more consistent splash time settings
* asks the user if he wants to delete the unencrypted file and delete it if YES
* manual: including hint, that the deleted file could be reconstructed from raw data on disk

Furthermore I have found a bug in keybox.c. If data is resetted in function "reset"
rb->gui_synclist_set_nb_items(&kb_list, 0);
must be called additionally to reset the number of items in kb_list too.
Comment by Gerhard Zintel (Gerbil) - Friday, 14 May 2010, 11:06 GMT
excluded the reset bug patch from above patch (rb->gui_synclist_set_nb_items(&kb_list, 0);). For that patch (11272) has been applied meanwhile into Rockbox I've deleted it from this ( FS#11260 ) one.