Bug 150605

Summary: kgpg does not remove temporal files after Encrypt folder
Product: [Applications] kgpg Reporter: Ana Guerrero <ana>
Component: generalAssignee: Jimmy Gilles <jimmygilles>
Status: RESOLVED FIXED    
Severity: normal CC: kde
Priority: NOR    
Version First Reported In: unspecified   
Target Milestone: ---   
Platform: Debian testing   
OS: Linux   
Latest Commit: Version Fixed/Implemented In:
Sentry Crash Report:
Attachments: Fix leak of KArchive instance
untested (!) patch against KDE 3.5

Description Ana Guerrero 2007-10-08 19:23:40 UTC 
Version:           1.2.2 (using KDE KDE 3.5.7)
Installed from:    Debian testing/unstable Packages
OS:                Linux

When encrypting a a folder with Kgpg via the option "Archive & Encrypt folder" in the context menu, it does not remove the temporal files *.tar created at /tmp/kde-user/file.tmp. This problem only happens if you compress with gzip or bzip2, not with zip.
Comment 1 Rolf Eike Beer 2007-11-26 11:58:05 UTC 
Created attachment 22203 [details]
Fix leak of KArchive instance

This fixed the leak of the KArchive instance used for encryption. This is
against current trunk. Please give it a test and report results.
Comment 2 Ana Guerrero 2007-11-26 23:57:58 UTC 
Hi Rolf, Thank you for your patch!

With "This is against current trunk" you meant this patch is for KDE4?
Comment 3 Rolf Eike Beer 2007-11-27 08:10:55 UTC 
Created attachment 22211 [details]
untested (!) patch against KDE 3.5

The same thing backported to KDE 3.5. Completely untested (i.e. not even
compile tested).
Comment 4 Ana Guerrero 2007-11-28 04:01:56 UTC 
The patch works perfectly against KDE 3.5.8 \o/
Thanks a lot.
Comment 5 Rolf Eike Beer 2007-11-28 08:25:08 UTC 
SVN commit 742520 by dakon:

Fix leak of KArchive instance when encrypting folder

This leads to leftover temporary files besides the memory leak.

BUG:150605

Port to trunk will follow soon, there are currently some additional problems.


 M  +2 -1      kgpg.cpp  
 M  +4 -1      kgpginterface.cpp  
 M  +3 -2      kgpginterface.h  


WebSVN link: http://websvn.kde.org/?view=rev&revision=742520
Comment 6 Rolf Eike Beer 2007-11-28 16:05:31 UTC 
SVN commit 742662 by dakon:

Fix leak of KArchive instance when encrypting folder 
 
This leads to leftover temporary files besides the memory leak. 
 
CCBUG:150605 
 
Encrypting folders still doesn't work but that's a completely unrelated 
problem.


 M  +2 -1      kgpg.cpp  
 M  +4 -1      kgpginterface.cpp  
 M  +5 -1      kgpginterface.h  


WebSVN link: http://websvn.kde.org/?view=rev&revision=742662