[prev in list] [next in list] [prev in thread] [next in thread] 

List:       koffice
Subject:    [Bug 54459] Security: backup files get different permissions
From:       David Faure <faure () kde ! org>
Date:       2004-05-11 14:47:37
Message-ID: 20040511144737.17898.qmail () ktown ! kde ! org
[Download RAW message or body]

------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
      
http://bugs.kde.org/show_bug.cgi?id=54459      
faure kde org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED



------- Additional Comments From faure kde org  2004-05-11 16:47 -------
CVS commit by faure: 

Oh well, I suck. Now this should fix 54459 again. I'll backport.
CCMAIL: 54459-done bugs kde org


  M +4 -4      koDocument.cc   1.308


--- koffice/lib/kofficecore/koDocument.cc  #1.307:1.308
 @ -48,7 +48,7  @
 #include <kmessagebox.h>
 #include <kdeversion.h>
+#include <kfileitem.h>
 #if ! KDE_IS_VERSION(3,1,90)
 #include <kdebugclasses.h>
-#include <kfileitem.h>
 #endif
 
 @ -383,9 +383,9  @ bool KoDocument::saveFile()
                 backup = d->m_backupPath +"/"+url().fileName();
             backup.setPath( backup.path() + QString::fromLatin1("~") );
-#if KDE_IS_VERSION(3,1,90)
-            KIO::NetAccess::file_copy( url(), backup, -1, true /*overwrite*/, false \
                /*resume*/, shells().current() );
-#else
             KFileItem item( entry, url() );
             Q_ASSERT( item.name() == url().fileName() );
+#if KDE_IS_VERSION(3,1,90)
+            KIO::NetAccess::file_copy( url(), backup, item.permissions(), true \
/*overwrite*/, false /*resume*/, shells().current() ); +#else
             KIO::NetAccess::del( backup ); // Copy does not remove existing \
destination file  KIO::NetAccess::copy( url(), backup );
____________________________________
koffice mailing list
koffice@mail.kde.org
To unsubscribe please visit:
https://mail.kde.org/mailman/listinfo/koffice


[prev in list] [next in list] [prev in thread] [next in thread]