The branch, master has been updated
       via  909056a s3-cli: fix bug 563, >8GB tar on BE machines
      from  cd3a935 s3: Fix a cut&paste error

http://gitweb.samba.org/?p=samba.git;a=shortlog;h=master


- Log -----------------------------------------------------------------
commit 909056a2daacd961b40158b86bc117650a897054
Author: Masafumi Nakayama <mas...@jp.ibm.com>
Date:   Wed Nov 2 10:35:19 2011 +0100

    s3-cli: fix bug 563, >8GB tar on BE machines
    
    Borrows on existing patches proposed by Craig Barratt and Brad Ellis.
    
    Signed-off-by: David Disseldorp <dd...@suse.de>
    
    Autobuild-User: David Disseldorp <dd...@samba.org>
    Autobuild-Date: Tue Jan  3 18:31:28 CET 2012 on sn-devel-104

-----------------------------------------------------------------------

Summary of changes:
 source3/client/clitar.c |   22 +++++++++++++++++-----
 1 files changed, 17 insertions(+), 5 deletions(-)


Changeset truncated at 500 lines:

diff --git a/source3/client/clitar.c b/source3/client/clitar.c
index 5943926..bf18e54 100644
--- a/source3/client/clitar.c
+++ b/source3/client/clitar.c
@@ -188,8 +188,10 @@ static void writetarheader(int f, const char *aname, 
uint64_t size, time_t mtime
 
                memset(hb.dbuf.size, 0, 4);
                hb.dbuf.size[0]=128;
-               for (i = 8, jp=(char*)&size; i; i--)
-                       hb.dbuf.size[i+3] = *(jp++);
+               for (i = 8; i; i--) {
+                       hb.dbuf.size[i+3] = size & 0xff;
+                       size >>= 8;
+               }
        }
        oct_it((uint64_t) mtime, 13, hb.dbuf.mtime);
        memcpy(hb.dbuf.chksum, "        ", sizeof(hb.dbuf.chksum));
@@ -291,7 +293,17 @@ of link other than a GNUtar Longlink - ignoring\n"));
        finfo->mtime_ts = finfo->ctime_ts =
                convert_time_t_to_timespec((time_t)strtol(hb->dbuf.mtime, NULL, 
8));
        finfo->atime_ts = convert_time_t_to_timespec(time(NULL));
-       finfo->size = unoct(hb->dbuf.size, sizeof(hb->dbuf.size));
+       if ((hb->dbuf.size[0] & 0xff) == 0x80) {
+               /* This is a non-POSIX compatible extention to extract files
+                       greater than 8GB. */
+               finfo->size = 0;
+               for (i = 0; i < 8; i++) {
+                       finfo->size <<= 8;
+                       finfo->size |= hb->dbuf.size[i+4] & 0xff;
+               }
+       } else {
+               finfo->size = unoct(hb->dbuf.size, sizeof(hb->dbuf.size));
+       }
 
        return True;
 }
@@ -1005,8 +1017,8 @@ static int skip_file(int skipsize)
 static int get_file(file_info2 finfo)
 {
        uint16_t fnum = (uint16_t) -1;
-       int pos = 0, dsize = 0, bpos = 0;
-       uint64_t rsize = 0;
+       int dsize = 0, bpos = 0;
+       uint64_t rsize = 0, pos = 0;
        NTSTATUS status;
 
        DEBUG(5, ("get_file: file: %s, size %.0f\n", finfo.name, 
(double)finfo.size));


-- 
Samba Shared Repository

Reply via email to