| -rw-r--r-- | library/filemanager.cpp | 167 | ||||
| -rw-r--r-- | library/filemanager.h | 6 |
2 files changed, 121 insertions, 52 deletions
diff --git a/library/filemanager.cpp b/library/filemanager.cpp index 2b97846..cc657fa 100644 --- a/library/filemanager.cpp +++ b/library/filemanager.cpp @@ -30,2 +30,7 @@ #include <stdlib.h> +#include <unistd.h> +#include <sys/stat.h> +#include <dirent.h> +#include <sys/sendfile.h> +#include <fcntl.h> @@ -58,7 +63,9 @@ bool FileManager::saveFile( const DocLnk &f, const QByteArray &data ) { - QString fn = f.file() + ".new"; + QString fn = f.file() + ".new"; ensurePathExists( fn ); QFile fl( fn ); - if ( !fl.open( IO_WriteOnly|IO_Raw ) ) - return FALSE; + if ( !fl.open( IO_WriteOnly|IO_Raw ) ) { + qWarning("open failed"); + return FALSE; + } int total_written = fl.writeBlock( data ); @@ -66,11 +73,11 @@ bool FileManager::saveFile( const DocLnk &f, const QByteArray &data ) if ( total_written != int(data.size()) || !f.writeLink() ) { - QFile::remove( fn ); - return FALSE; + QFile::remove( fn ); + return FALSE; } + qDebug("total written %d out of %d", total_written, data.size()); // else rename the file... - if ( ::rename( fn.latin1(), f.file().latin1() ) < 0 ) { - qWarning( "problem renaming file %s to %s, errno: %d", fn.latin1(), - f.file().latin1(), errno ); - // remove the file... - QFile::remove( fn ); + if ( !renameFile( fn.latin1(), f.file().latin1() ) ) { + qWarning( "problem renaming file %s to %s, errno: %d", fn.latin1(), + f.file().latin1(), errno ); + // remove the file... } @@ -88,3 +95,3 @@ bool FileManager::saveFile( const DocLnk &f, const QString &text ) { - QString fn = f.file() + ".new"; + QString fn = f.file() + ".new"; ensurePathExists( fn ); @@ -92,4 +99,4 @@ bool FileManager::saveFile( const DocLnk &f, const QString &text ) if ( !fl.open( IO_WriteOnly|IO_Raw ) ) { - qDebug( "open failed: %s", fn.latin1() ); - return FALSE; + qWarning("open failed"); + return FALSE; } @@ -101,11 +108,10 @@ bool FileManager::saveFile( const DocLnk &f, const QString &text ) if ( total_written != int(cstr.length()) || !f.writeLink() ) { - QFile::remove( fn ); - return FALSE; + QFile::remove( fn ); + return FALSE; } - // okay now rename the file... - if ( ::rename( fn.latin1(), f.file().latin1() ) < 0 ) { - qWarning( "problem renaming file %s to %s, errno: %d", fn.latin1(), - f.file().latin1(), errno ); - // remove the tmp file, otherwise, it will just lay around... - QFile::remove( fn.latin1() ); + // okay now rename the file.. + if ( !renameFile( fn.latin1(), f.file().latin1() ) ) { + qWarning( "problem renaming file %s to %s, errno: %d", fn.latin1(), + f.file().latin1(), errno ); + } @@ -127,3 +133,3 @@ bool FileManager::loadFile( const DocLnk &f, QString &text ) if ( !fl.open( IO_ReadOnly ) ) - return FALSE; + return FALSE; QTextStream ts( &fl ); @@ -151,6 +157,6 @@ bool FileManager::loadFile( const DocLnk &f, QByteArray &ba ) if ( !fl.open( IO_ReadOnly ) ) - return FALSE; + return FALSE; ba.resize( fl.size() ); if ( fl.size() > 0 ) - fl.readBlock( ba.data(), fl.size() ); + fl.readBlock( ba.data(), fl.size() ); fl.close(); @@ -169,3 +175,3 @@ bool FileManager::copyFile( const AppLnk &src, const AppLnk &dest ) if ( !sf.open( IO_ReadOnly ) ) - return FALSE; + return FALSE; @@ -175,3 +181,3 @@ bool FileManager::copyFile( const AppLnk &src, const AppLnk &dest ) if ( !df.open( IO_WriteOnly|IO_Raw ) ) - return FALSE; + return FALSE; @@ -182,12 +188,12 @@ bool FileManager::copyFile( const AppLnk &src, const AppLnk &dest ) while ( ok && !sf.atEnd() ) { - bytesRead = sf.readBlock( buffer, bufsize ); - if ( bytesRead < 0 ) - ok = FALSE; - while ( ok && bytesRead > 0 ) { - int bytesWritten = df.writeBlock( buffer, bytesRead ); - if ( bytesWritten < 0 ) - ok = FALSE; - else - bytesRead -= bytesWritten; - } + bytesRead = sf.readBlock( buffer, bufsize ); + if ( bytesRead < 0 ) + ok = FALSE; + while ( ok && bytesRead > 0 ) { + int bytesWritten = df.writeBlock( buffer, bytesRead ); + if ( bytesWritten < 0 ) + ok = FALSE; + else + bytesRead -= bytesWritten; + } } @@ -195,14 +201,14 @@ bool FileManager::copyFile( const AppLnk &src, const AppLnk &dest ) if ( ok ) - ok = dest.writeLink(); + ok = dest.writeLink(); if ( ok ) { - // okay now rename the file... - if ( ::rename( fn.latin1(), dest.file().latin1() ) < 0 ) { - qWarning( "problem renaming file %s to %s, errno: %d", fn.latin1(), - dest.file().latin1(), errno ); - // remove the tmp file, otherwise, it will just lay around... - QFile::remove( fn.latin1() ); - } + // okay now rename the file... + if ( !renameFile( fn.latin1(), dest.file().latin1() ) ) { + qWarning( "problem renaming file %s to %s, errno: %d", fn.latin1(), + dest.file().latin1(), errno ); + // remove the tmp file, otherwise, it will just lay around... + QFile::remove( fn.latin1() ); + } } else { - QFile::remove( fn.latin1() ); + QFile::remove( fn.latin1() ); } @@ -212,2 +218,63 @@ bool FileManager::copyFile( const AppLnk &src, const AppLnk &dest ) +bool FileManager::copyFile( const QString & src, const QString & dest ) { + bool success = true; + struct stat status; + int read_fd=0; + int write_fd=0; + struct stat stat_buf; + off_t offset = 0; + QFile srcFile(src); + QFile destFile(dest); + + if(!srcFile.open( IO_ReadOnly|IO_Raw)) { + return success = false; + } + read_fd = srcFile.handle(); + if(read_fd != -1) { + fstat (read_fd, &stat_buf); + if( !destFile.open( IO_WriteOnly|IO_Raw ) ) + return success = false; + write_fd = destFile.handle(); + if(write_fd != -1) { + int err=0; + QString msg; + err = sendfile(write_fd, read_fd, &offset, stat_buf.st_size); + if( err == -1) { + switch(err) { + case EBADF : msg = "The input file was not opened for reading or the output file was not opened for writing. "; + case EINVAL: msg = "Descriptor is not valid or locked. "; + case ENOMEM: msg = "Insufficient memory to read from in_fd."; + case EIO: msg = "Unspecified error while reading from in_fd."; + }; + success = false; + } + } else { + qWarning("open write failed %s, %s",src.latin1(), dest.latin1()); + success = false; + } + } else { + qWarning("open read failed %s, %s",src.latin1(), dest.latin1()); + success = false; + } + srcFile.close(); + destFile.close(); + // Set file permissions + if( stat( (const char *) src, &status ) == 0 ) { + chmod( (const char *) dest, status.st_mode ); + } + + return success; +} + + +bool FileManager::renameFile( const QString & src, const QString & dest ) { + if(copyFile( src, dest )) { + if(QFile::remove(src) ) { + return true; + } + } + return false; +} + + /*! @@ -223,3 +290,3 @@ QIODevice* FileManager::openFile( const DocLnk& f ) if ( !fl->open( IO_ReadOnly ) ) { - delete fl; + delete fl; fl = 0; @@ -241,5 +308,5 @@ QIODevice* FileManager::saveFile( const DocLnk& f ) if ( fl->open( IO_WriteOnly ) ) { - f.writeLink(); + f.writeLink(); } else { - delete fl; + delete fl; fl = 0; @@ -268,4 +335,4 @@ bool FileManager::ensurePathExists( const QString &fn ) if ( !fi.exists() ) { - if ( system(("mkdir -p "+fi.filePath())) ) - return FALSE; + if ( system(("mkdir -p "+fi.filePath())) ) + return FALSE; } diff --git a/library/filemanager.h b/library/filemanager.h index f8d9425..61a3341 100644 --- a/library/filemanager.h +++ b/library/filemanager.h @@ -40,4 +40,6 @@ public: bool copyFile( const AppLnk &src, const AppLnk &dest ); - - // The caller must delete the return values. + bool copyFile( const QString & src, const QString & dest ); + bool renameFile( const QString &, const QString &); + +// The caller must delete the return values. QIODevice* openFile( const DocLnk& ); |