Labels Milestones

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

Sloučený

MicheleC sloučil 1 commity z větve feat/tdewallet-access-R14-0-6 do větve r14.0.x před před 5 roky

Konverzace 2 Commity 1 Změněné soubory 1

MicheleC okomentoval před 5 roky

Vlastník

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

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

MicheleC přidal/a toto do milníku R14.0.6 release před 5 roky

SlavekB okomentoval před 5 roky

Vlastník

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 okomentoval před 5 roky

Autor

Vlastník

Great! I think we are ready to merge then

Great! I think we are ready to merge then

MicheleC uzavřel/a tento požadavek na natažení před 5 roky

MicheleC odstranil/a větev feat/tdewallet-access-R14-0-6 před 5 roky

Požadavek na natažení byl sloučen jako 987768c54d.

Přihlaste se pro zapojení do konverzace.

Posuzovatelé

Požádat o posouzení

Žádní posuzovatelé

Štítky

Použít štítky

Zrušit štítky

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

Bez štítku 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

Milník

Nastavit milník

Smazat milník

Žádné položky

Bez milníku

R14.0.6 release

Zpracovatelé

Přiřadit uživatele

Smazat zpracovatele

Bez zpracovatelů

2 účastníků

Oznámení

Odebírat

Termín dokončení

Termín dokončení není platný nebo je mimo rozsah. Použijte prosím formát „rrrr-mm-dd“.

Žádný termín dokončení.

Závislosti

Nejsou nastaveny žádné závislosti.

Reference: TDE/tdelibs#16

Zapsat Náhled

Načítá se…

Zrušit

Uložit

Není zde žádný obsah.