TDE
/
tdelibs
7
1
Fork 0
Code Issues 16 Pull-Requests 8 Releases Wiki Aktivität
Label Meilensteine

Fix to access old TDE wallet with special characters (R14.0.x series) #16

Zusammengeführt
MicheleC hat 1 Commits von feat/tdewallet-access-R14-0-6 nach r14.0.x vor 5 Jahren zusammengeführt
Diskussion 2 Commits 1 Geänderte Dateien 1
MicheleC hat vor 5 Jahren kommentiert
Besitzer

Same as #15 but for R14.0.x series

Same as #15 but for R14.0.x series
MicheleC hat diesen Issue vor 5 Jahren zum R14.0.6 release Meilenstein hinzugefügt
SlavekB hat vor 5 Jahren kommentiert
Besitzer

Problem with previous patch version identified and resolved. This was a similar problem as we recently dealt with in KPasswordEdit::password().

Calling TQString::fromLocal8Bit( "..." ) creates a temporary TQString object, but the subsequent use of const char * obtained by the ascii() method then refers to a portion of memory that has already been released and used for another purpose.

With a modified version of the patch, opening a wallet with a password with special characters works also in R14.0.6.

Problem with previous patch version identified and resolved. This was a similar problem as we recently dealt with in `KPasswordEdit::password()`. Calling `TQString::fromLocal8Bit( "..." )` creates a temporary TQString object, but the subsequent use of `const char *` obtained by the `ascii()` method then refers to a portion of memory that has already been released and used for another purpose. With a modified version of the patch, opening a wallet with a password with special characters works also in R14.0.6.
MicheleC hat vor 5 Jahren kommentiert
Ersteller
Besitzer

Great! I think we are ready to merge then

Great! I think we are ready to merge then
MicheleC hat diesen Pull-Request vor 5 Jahren geschlossen
MicheleC löschte die Branch feat/tdewallet-access-R14-0-6 vor 5 Jahren
Der Pull Request wurde als 987768c54d gemergt.
Anmelden, um an der Diskussion teilzunehmen.
Reviewer
Überprüfung anfordern
Keine Reviewer
Label
Label anwenden
Label entfernen
GE/need-info
General - need additional info from contributor PR/keep-branch
Pull request - do not delete branch after merging PR/not-ok
Pull request - need fixing PR/rfc
Pull request - request for comments PR/update-trans
Pull request - update to translation files needed PR/wip
Pull request - work in progress RS/R14.0.x
Related to R14.0.x series RS/R14.1.x
Related to R14.1.x series SL/critical
Severity level - critical SL/major
Severity level - major SL/minor
Severity level - minor SL/normal
Severity level - normal SL/regression
Severity level - regression from previous version SL/trivial
Severity level - trivial SL/wishlist
Severity level - wishlist request ST/duplicate
Status - duplicate of another issue ST/invalid
Status - invalid report ST/notourproblem
Status - not our problem ST/rejected
Status - rejected ST/wontfix
Status - won't fix ST/worksforme
Status - works for me, unable to reproduce
Kein Label GE/need-info PR/keep-branch PR/not-ok PR/rfc PR/update-trans PR/wip RS/R14.0.x RS/R14.1.x SL/critical SL/major SL/minor SL/normal SL/regression SL/trivial SL/wishlist ST/duplicate ST/invalid ST/notourproblem ST/rejected ST/wontfix ST/worksforme
Meilenstein
Meilenstein setzen
Meilenstein entfernen
Keine Einträge
Kein Meilenstein
R14.0.6 release
Zuständig
Benutzer zuweisen
Zuständige entfernen
Niemand zuständig
2 Beteiligte
Nachrichten
Fällig am

Kein Fälligkeitsdatum gesetzt.

Abhängigkeiten

Keine Abhängigkeiten gesetzt.

Referenz: TDE/tdelibs#16
Verfassen Vorschau
Laden…
Abbrechen
Speichern
Hier gibt es bis jetzt noch keinen Inhalt.
Branch „feat/tdewallet-access-R14-0-6“ löschen

Das Löschen eines Branches ist permanent. Es KANN NICHT rückgängig gemacht werden. Fortfahren?

Abbrechen
Ja