]> git.sur5r.net Git - bacula/bacula/blobdiff - bacula/src/findlib/attribs.c
Apply Preben 'Peppe' Guldberg <peppe@wielders.org>
[bacula/bacula] / bacula / src / findlib / attribs.c
index 99237457fbeaef06d8d45bc44c914ac5f60ca061..ad867702f87a20dd9aad7039c553d096fad4ebf3 100755 (executable)
@@ -9,7 +9,7 @@
  *
  */
 /*
-   Copyright (C) 2000-2004 Kern Sibbald and John Walker
+   Copyright (C) 2002-2004 Kern Sibbald and John Walker
 
    This program is free software; you can redistribute it and/or
    modify it under the terms of the GNU General Public License as
@@ -34,7 +34,7 @@
 #if defined(HAVE_CYGWIN) || defined(HAVE_WIN32)
 
 /* Forward referenced subroutines */
-static int set_win32_attributes(JCR *jcr, ATTR *attr, BFILE *ofd);
+static bool set_win32_attributes(JCR *jcr, ATTR *attr, BFILE *ofd);
 void unix_name_to_win32(POOLMEM **win32_name, char *name);
 void win_error(JCR *jcr, char *prefix, POOLMEM *ofile);
 HANDLE bget_handle(BFILE *bfd);
@@ -52,7 +52,7 @@ HANDLE bget_handle(BFILE *bfd);
 /*=============================================================*/
 
 /*
- * Return the data stream that will be used 
+ * Return the data stream that will be used
  */
 int select_data_stream(FF_PKT *ff_pkt)
 {
@@ -82,8 +82,8 @@ int select_data_stream(FF_PKT *ff_pkt)
 }
 
 
-/* 
- * Encode a stat structure into a base64 character string   
+/*
+ * Encode a stat structure into a base64 character string
  *   All systems must create such a structure.
  *   In addition, we tack on the LinkFI, which is non-zero in
  *   the case of a hard linked file that has no data.  This
@@ -117,10 +117,17 @@ void encode_stat(char *buf, FF_PKT *ff_pkt, int data_stream)
    *p++ = ' ';
    p += to_base64((int64_t)statp->st_size, p);
    *p++ = ' ';
+#ifndef HAVE_MINGW
    p += to_base64((int64_t)statp->st_blksize, p);
    *p++ = ' ';
    p += to_base64((int64_t)statp->st_blocks, p);
    *p++ = ' ';
+#else
+   p += to_base64((int64_t)0, p); /* output place holder */
+   *p++ = ' ';
+   p += to_base64((int64_t)0, p); /* output place holder */
+   *p++ = ' ';
+#endif
    p += to_base64((int64_t)statp->st_atime, p);
    *p++ = ' ';
    p += to_base64((int64_t)statp->st_mtime, p);
@@ -143,51 +150,68 @@ void encode_stat(char *buf, FF_PKT *ff_pkt, int data_stream)
 }
 
 
+/* Do casting according to unknown type to keep compiler happy */
+#if !HAVE_GCC & HAVE_SUN_OS
+#define plug(st, val) st = val       /* brain damaged compiler */
+#else
+template <class T> void plug(T &st, uint64_t val)
+    { st = static_cast<T>(val); }
+#endif
+
 
 /* Decode a stat packet from base64 characters */
-int decode_stat(char *buf, struct stat *statp, int32_t *LinkFI) 
+int decode_stat(char *buf, struct stat *statp, int32_t *LinkFI)
 {
    char *p = buf;
    int64_t val;
 
    p += from_base64(&val, p);
-   statp->st_dev = val;
-   p++;                              /* skip space */
+   plug(statp->st_dev, val);
+   p++;
    p += from_base64(&val, p);
-   statp->st_ino = val;
+   plug(statp->st_ino, val);
    p++;
    p += from_base64(&val, p);
-   statp->st_mode = val;
+   plug(statp->st_mode, val);
    p++;
    p += from_base64(&val, p);
-   statp->st_nlink = val;
+   plug(statp->st_nlink, val);
    p++;
    p += from_base64(&val, p);
-   statp->st_uid = val;
+   plug(statp->st_uid, val);
    p++;
    p += from_base64(&val, p);
-   statp->st_gid = val;
+   plug(statp->st_gid, val);
    p++;
    p += from_base64(&val, p);
-   statp->st_rdev = val;
+   plug(statp->st_rdev, val);
    p++;
    p += from_base64(&val, p);
-   statp->st_size = val;
+   plug(statp->st_size, val);
    p++;
+#ifndef HAVE_MINGW
    p += from_base64(&val, p);
-   statp->st_blksize = val;
+   plug(statp->st_blksize, val);
    p++;
    p += from_base64(&val, p);
-   statp->st_blocks = val;
+   plug(statp->st_blocks, val);
    p++;
+#else
    p += from_base64(&val, p);
-   statp->st_atime = val;
+//   plug(statp->st_blksize, val);
    p++;
    p += from_base64(&val, p);
-   statp->st_mtime = val;
+//   plug(statp->st_blocks, val);
    p++;
+#endif
    p += from_base64(&val, p);
-   statp->st_ctime = val;
+   plug(statp->st_atime, val);
+   p++;
+   p += from_base64(&val, p);
+   plug(statp->st_mtime, val);
+   p++;
+   p += from_base64(&val, p);
+   plug(statp->st_ctime, val);
 
    /* Optional FileIndex of hard linked file data */
    if (*p == ' ' || (*p != 0 && *(p+1) == ' ')) {
@@ -204,12 +228,12 @@ int decode_stat(char *buf, struct stat *statp, int32_t *LinkFI)
       p++;
       p += from_base64(&val, p);
 #ifdef HAVE_CHFLAGS
-      statp->st_flags = (uint32_t)val;
+      plug(statp->st_flags, val);
    } else {
       statp->st_flags  = 0;
 #endif
    }
-    
+
    /* Look for data stream id */
    if (*p == ' ' || (*p != 0 && *(p+1) == ' ')) {
       p++;
@@ -231,7 +255,7 @@ int32_t decode_LinkFI(char *buf, struct stat *statp)
    skip_nonspaces(&p);               /* st_ino */
    p++;
    p += from_base64(&val, p);
-   statp->st_mode = val;             /* st_mode */
+   plug(statp->st_mode, val);        /* st_mode */
    p++;
    skip_nonspaces(&p);               /* st_nlink */
    p++;
@@ -265,96 +289,115 @@ int32_t decode_LinkFI(char *buf, struct stat *statp)
 /*
  * Set file modes, permissions and times
  *
- *  fname is the original filename  
+ *  fname is the original filename
  *  ofile is the output filename (may be in a different directory)
  *
- * Returns:  1 on success
- *          0 on failure
+ * Returns:  true  on success
+ *          false on failure
  */
-int set_attributes(JCR *jcr, ATTR *attr, BFILE *ofd)
+bool set_attributes(JCR *jcr, ATTR *attr, BFILE *ofd)
 {
-   struct utimbuf ut;   
+   struct utimbuf ut;
    mode_t old_mask;
-   int stat = 1;
+   bool ok = true;
+   off_t fsize;
 
 #if defined(HAVE_CYGWIN) || defined(HAVE_WIN32)
    if (attr->stream == STREAM_UNIX_ATTRIBUTES_EX &&
        set_win32_attributes(jcr, attr, ofd)) {
        if (is_bopen(ofd)) {
-          bclose(ofd); 
+          bclose(ofd);
        }
-       pm_strcpy(&attr->ofname, "*none*");
-       return 1;
+       pm_strcpy(attr->ofname, "*none*");
+       return true;
    }
    if (attr->data_stream == STREAM_WIN32_DATA ||
        attr->data_stream == STREAM_WIN32_GZIP_DATA) {
       if (is_bopen(ofd)) {
-        bclose(ofd); 
+        bclose(ofd);
       }
-      pm_strcpy(&attr->ofname, "*none*");
-      return 1;
+      pm_strcpy(attr->ofname, "*none*");
+      return true;
    }
 
 
    /*
     * If Windows stuff failed, e.g. attempt to restore Unix file
-    *  to Windows, simply fall through and we will do it the    
+    *  to Windows, simply fall through and we will do it the
     *  universal way.
     */
 #endif
 
    old_mask = umask(0);
    if (is_bopen(ofd)) {
+      char ec1[50], ec2[50];
+      fsize = blseek(ofd, 0, SEEK_CUR);
       bclose(ofd);                   /* first close file */
+      if (fsize > 0 && fsize != attr->statp.st_size) {
+        Jmsg3(jcr, M_ERROR, 0, _("File size of restored file %s not correct. Original %s, restored %s.\n"),
+           attr->ofname, edit_uint64(attr->statp.st_size, ec1),
+           edit_uint64(fsize, ec2));
+      }
    }
 
    ut.actime = attr->statp.st_atime;
    ut.modtime = attr->statp.st_mtime;
 
    /* ***FIXME**** optimize -- don't do if already correct */
-   /* 
+   /*
     * For link, change owner of link using lchown, but don't
     *  try to do a chmod as that will update the file behind it.
     */
    if (attr->type == FT_LNK) {
       /* Change owner of link, not of real file */
       if (lchown(attr->ofname, attr->statp.st_uid, attr->statp.st_gid) < 0) {
-         Jmsg2(jcr, M_ERROR, 0, _("Unable to set file owner %s: ERR=%s\n"),
-           attr->ofname, strerror(errno));
-        stat = 0;
+        berrno be;
+        Jmsg2(jcr, M_ERROR, 0, _("Unable to set file owner %s: ERR=%s\n"),
+           attr->ofname, be.strerror());
+        ok = false;
       }
    } else {
       if (chown(attr->ofname, attr->statp.st_uid, attr->statp.st_gid) < 0) {
-         Jmsg2(jcr, M_ERROR, 0, _("Unable to set file owner %s: ERR=%s\n"),
-           attr->ofname, strerror(errno));
-        stat = 0;
+        berrno be;
+        Jmsg2(jcr, M_ERROR, 0, _("Unable to set file owner %s: ERR=%s\n"),
+           attr->ofname, be.strerror());
+        ok = false;
       }
       if (chmod(attr->ofname, attr->statp.st_mode) < 0) {
-         Jmsg2(jcr, M_ERROR, 0, _("Unable to set file modes %s: ERR=%s\n"),
-           attr->ofname, strerror(errno));
-        stat = 0;
+        berrno be;
+        Jmsg2(jcr, M_ERROR, 0, _("Unable to set file modes %s: ERR=%s\n"),
+           attr->ofname, be.strerror());
+        ok = false;
       }
 
-      /* FreeBSD user flags */
-#ifdef HAVE_CHFLAGS
-      if (chflags(attr->ofname, attr->statp.st_flags) < 0) {
-         Jmsg2(jcr, M_ERROR, 0, _("Unable to set file flags %s: ERR=%s\n"),
-           attr->ofname, strerror(errno));
-        stat = 0;
-      }
-#endif
       /*
        * Reset file times.
        */
       if (utime(attr->ofname, &ut) < 0) {
-         Jmsg2(jcr, M_ERROR, 0, _("Unable to set file times %s: ERR=%s\n"),
-           attr->ofname, strerror(errno));
-        stat = 0;
+        berrno be;
+        Jmsg2(jcr, M_ERROR, 0, _("Unable to set file times %s: ERR=%s\n"),
+           attr->ofname, be.strerror());
+        ok = false;
+      }
+#ifdef HAVE_CHFLAGS
+      /*
+       * FreeBSD user flags
+       *
+       * Note, this should really be done before the utime() above,
+       *  but if the immutable bit is set, it will make the utimes()
+       *  fail.
+       */
+      if (chflags(attr->ofname, attr->statp.st_flags) < 0) {
+        berrno be;
+        Jmsg2(jcr, M_ERROR, 0, _("Unable to set file flags %s: ERR=%s\n"),
+           attr->ofname, be.strerror());
+        ok = false;
       }
+#endif
    }
-   pm_strcpy(&attr->ofname, "*none*");
+   pm_strcpy(attr->ofname, "*none*");
    umask(old_mask);
-   return stat;
+   return ok;
 }
 
 
@@ -365,7 +408,7 @@ int set_attributes(JCR *jcr, ATTR *attr, BFILE *ofd)
 /*=============================================================*/
 
 #if !defined(HAVE_CYGWIN) && !defined(HAVE_WIN32)
-    
+
 /*
  * It is possible to piggyback additional data e.g. ACLs on
  *   the encode_stat() data by returning the extended attributes
@@ -377,7 +420,16 @@ int set_attributes(JCR *jcr, ATTR *attr, BFILE *ofd)
  */
 int encode_attribsEx(JCR *jcr, char *attribsEx, FF_PKT *ff_pkt)
 {
+#ifdef HAVE_DARWIN_OS
+   char *p;
+   p = attribsEx;
+   if (ff_pkt->flags & FO_HFSPLUS) {
+      p += to_base64((uint64_t)(ff_pkt->hfsinfo.rsrclength), p);
+   }
+   *p = 0;
+#else
    *attribsEx = 0;                   /* no extended attributes */
+#endif
    return STREAM_UNIX_ATTRIBUTES;
 }
 
@@ -401,7 +453,7 @@ int encode_attribsEx(JCR *jcr, char *attribsEx, FF_PKT *ff_pkt)
 
    attribsEx[0] = 0;                 /* no extended attributes */
 
-   if (!p_GetFileAttributesEx) {                                
+   if (!p_GetFileAttributesEx) {
       return STREAM_UNIX_ATTRIBUTES;
    }
 
@@ -435,26 +487,26 @@ int encode_attribsEx(JCR *jcr, char *attribsEx, FF_PKT *ff_pkt)
 
 /* Define attributes that are legal to set with SetFileAttributes() */
 #define SET_ATTRS ( \
-         FILE_ATTRIBUTE_ARCHIVE| \
-         FILE_ATTRIBUTE_HIDDEN| \
-         FILE_ATTRIBUTE_NORMAL| \
-         FILE_ATTRIBUTE_NOT_CONTENT_INDEXED| \
-         FILE_ATTRIBUTE_OFFLINE| \
-         FILE_ATTRIBUTE_READONLY| \
-         FILE_ATTRIBUTE_SYSTEM| \
+        FILE_ATTRIBUTE_ARCHIVE| \
+        FILE_ATTRIBUTE_HIDDEN| \
+        FILE_ATTRIBUTE_NORMAL| \
+        FILE_ATTRIBUTE_NOT_CONTENT_INDEXED| \
+        FILE_ATTRIBUTE_OFFLINE| \
+        FILE_ATTRIBUTE_READONLY| \
+        FILE_ATTRIBUTE_SYSTEM| \
         FILE_ATTRIBUTE_TEMPORARY)
 
 
 /*
  * Set Extended File Attributes for Win32
  *
- *  fname is the original filename  
+ *  fname is the original filename
  *  ofile is the output filename (may be in a different directory)
  *
- * Returns:  1 on success
- *          0 on failure
+ * Returns:  true  on success
+ *          false on failure
  */
-static int set_win32_attributes(JCR *jcr, ATTR *attr, BFILE *ofd)
+static bool set_win32_attributes(JCR *jcr, ATTR *attr, BFILE *ofd)
 {
    char *p = attr->attrEx;
    int64_t val;
@@ -462,8 +514,8 @@ static int set_win32_attributes(JCR *jcr, ATTR *attr, BFILE *ofd)
    ULARGE_INTEGER li;
    POOLMEM *win32_ofile;
 
-   if (!p_GetFileAttributesEx) {                                
-      return 0;
+   if (!p_GetFileAttributesEx) {
+      return false;
    }
 
    if (!p || !*p) {                  /* we should have attributes */
@@ -471,13 +523,13 @@ static int set_win32_attributes(JCR *jcr, ATTR *attr, BFILE *ofd)
       if (is_bopen(ofd)) {
         bclose(ofd);
       }
-      return 0;
+      return false;
    } else {
       Dmsg2(100, "Attribs %s = %s\n", attr->ofname, attr->attrEx);
    }
 
    p += from_base64(&val, p);
-   atts.dwFileAttributes = val;
+   plug(atts.dwFileAttributes, val);
    p++;                              /* skip space */
    p += from_base64(&val, p);
    li.QuadPart = val;
@@ -493,12 +545,12 @@ static int set_win32_attributes(JCR *jcr, ATTR *attr, BFILE *ofd)
    li.QuadPart = val;
    atts.ftLastWriteTime.dwLowDateTime = li.LowPart;
    atts.ftLastWriteTime.dwHighDateTime = li.HighPart;
-   p++;   
+   p++;
    p += from_base64(&val, p);
-   atts.nFileSizeHigh = val;
+   plug(atts.nFileSizeHigh, val);
    p++;
    p += from_base64(&val, p);
-   atts.nFileSizeLow = val;
+   plug(atts.nFileSizeLow, val);
 
    /* Convert to Windows path format */
    win32_ofile = get_pool_memory(PM_FNAME);
@@ -519,7 +571,7 @@ static int set_win32_attributes(JCR *jcr, ATTR *attr, BFILE *ofd)
                         &atts.ftCreationTime,
                         &atts.ftLastAccessTime,
                         &atts.ftLastWriteTime)) {
-         win_error(jcr, "SetFileTime:", win32_ofile);
+        win_error(jcr, "SetFileTime:", win32_ofile);
       }
       bclose(ofd);
    }
@@ -527,11 +579,11 @@ static int set_win32_attributes(JCR *jcr, ATTR *attr, BFILE *ofd)
    Dmsg1(100, "SetFileAtts %s\n", attr->ofname);
    if (!(atts.dwFileAttributes & FILE_ATTRIBUTE_DIRECTORY)) {
       if (!SetFileAttributes(win32_ofile, atts.dwFileAttributes & SET_ATTRS)) {
-         win_error(jcr, "SetFileAttributes:", win32_ofile);
+        win_error(jcr, "SetFileAttributes:", win32_ofile);
       }
    }
    free_pool_memory(win32_ofile);
-   return 1;
+   return true;
 }
 
 void win_error(JCR *jcr, char *prefix, POOLMEM *win32_ofile)