Blame SOURCES/unzip-6.0-close.patch

02d1f3
diff -up unzip60/extract.c.close unzip60/extract.c
02d1f3
--- unzip60/extract.c.close	2009-03-14 02:32:52.000000000 +0100
02d1f3
+++ unzip60/extract.c	2009-11-19 08:17:23.481263496 +0100
02d1f3
@@ -1924,24 +1924,21 @@ static int extract_or_test_member(__G)  
02d1f3
 
02d1f3
 #ifdef VMS                  /* VMS:  required even for stdout! (final flush) */
02d1f3
     if (!uO.tflag)           /* don't close NULL file */
02d1f3
-        close_outfile(__G);
02d1f3
+        error = close_outfile(__G);
02d1f3
 #else
02d1f3
 #ifdef DLL
02d1f3
     if (!uO.tflag && (!uO.cflag || G.redirect_data)) {
02d1f3
         if (G.redirect_data)
02d1f3
             FINISH_REDIRECT();
02d1f3
         else
02d1f3
-            close_outfile(__G);
02d1f3
+            error = close_outfile(__G);
02d1f3
     }
02d1f3
 #else
02d1f3
     if (!uO.tflag && !uO.cflag)   /* don't close NULL file or stdout */
02d1f3
-        close_outfile(__G);
02d1f3
+        error = close_outfile(__G);
02d1f3
 #endif
02d1f3
 #endif /* VMS */
02d1f3
 
02d1f3
-            /* GRR: CONVERT close_outfile() TO NON-VOID:  CHECK FOR ERRORS! */
02d1f3
-
02d1f3
-
02d1f3
     if (G.disk_full) {            /* set by flush() */
02d1f3
         if (G.disk_full > 1) {
02d1f3
 #if (defined(DELETE_IF_FULL) && defined(HAVE_UNLINK))
02d1f3
diff -up unzip60/unix/unix.c.close unzip60/unix/unix.c
02d1f3
--- unzip60/unix/unix.c.close	2009-01-24 00:31:26.000000000 +0100
02d1f3
+++ unzip60/unix/unix.c	2009-11-19 08:33:25.568389171 +0100
02d1f3
@@ -1096,10 +1096,41 @@ static int get_extattribs(__G__ pzt, z_u
02d1f3
 #ifndef MTS
02d1f3
 
02d1f3
 /****************************/
02d1f3
+/* Function CloseError()    */
02d1f3
+/***************************/
02d1f3
+
02d1f3
+int CloseError(__G)
02d1f3
+    __GDEF
02d1f3
+{
02d1f3
+    int errval = PK_OK;
02d1f3
+    
02d1f3
+    if (fclose(G.outfile) < 0) {
02d1f3
+          switch (errno) {
02d1f3
+                case ENOSPC:
02d1f3
+                    /* Do we need this on fileio.c? */
02d1f3
+                    Info(slide, 0x4a1, ((char *)slide, "%s: write error (disk full?).   Continue? (y/n/^C) ",
02d1f3
+                          FnFilter1(G.filename)));
02d1f3
+                    fgets(G.answerbuf, 9, stdin);
02d1f3
+                    if (*G.answerbuf == 'y')     /* stop writing to this file */
02d1f3
+                        G.disk_full = 1;         /* pass to next */
02d1f3
+                    else
02d1f3
+                        G.disk_full = 2;         /* no: exit program */
02d1f3
+          
02d1f3
+                    errval = PK_DISK;
02d1f3
+                    break;
02d1f3
+
02d1f3
+                default:
02d1f3
+                    errval = PK_WARN;
02d1f3
+          }
02d1f3
+     }
02d1f3
+     return errval;
02d1f3
+} /* End of CloseError() */
02d1f3
+
02d1f3
+/****************************/
02d1f3
 /* Function close_outfile() */
02d1f3
 /****************************/
02d1f3
 
02d1f3
-void close_outfile(__G)    /* GRR: change to return PK-style warning level */
02d1f3
+int close_outfile(__G) 
02d1f3
     __GDEF
02d1f3
 {
02d1f3
     union {
02d1f3
@@ -1108,6 +1139,7 @@ void close_outfile(__G)    /* GRR: chang
02d1f3
     } zt;
02d1f3
     ulg z_uidgid[2];
02d1f3
     int have_uidgid_flg;
02d1f3
+    int errval = PK_OK;
02d1f3
 
02d1f3
     have_uidgid_flg = get_extattribs(__G__ &(zt.t3), z_uidgid);
02d1f3
 
02d1f3
@@ -1141,16 +1173,16 @@ void close_outfile(__G)    /* GRR: chang
02d1f3
             Info(slide, 0x201, ((char *)slide,
02d1f3
               "warning:  symbolic link (%s) failed: mem alloc overflow\n",
02d1f3
               FnFilter1(G.filename)));
02d1f3
-            fclose(G.outfile);
02d1f3
-            return;
02d1f3
+            errval = CloseError(G.outfile, G.filename);
02d1f3
+            return errval ? errval : PK_WARN;
02d1f3
         }
02d1f3
 
02d1f3
         if ((slnk_entry = (slinkentry *)malloc(slnk_entrysize)) == NULL) {
02d1f3
             Info(slide, 0x201, ((char *)slide,
02d1f3
               "warning:  symbolic link (%s) failed: no mem\n",
02d1f3
               FnFilter1(G.filename)));
02d1f3
-            fclose(G.outfile);
02d1f3
-            return;
02d1f3
+            errval = CloseError(G.outfile, G.filename);
02d1f3
+            return errval ? errval : PK_WARN;
02d1f3
         }
02d1f3
         slnk_entry->next = NULL;
02d1f3
         slnk_entry->targetlen = ucsize;
02d1f3
@@ -1174,10 +1206,10 @@ void close_outfile(__G)    /* GRR: chang
02d1f3
               "warning:  symbolic link (%s) failed\n",
02d1f3
               FnFilter1(G.filename)));
02d1f3
             free(slnk_entry);
02d1f3
-            fclose(G.outfile);
02d1f3
-            return;
02d1f3
+            errval = CloseError(G.outfile, G.filename);
02d1f3
+            return errval ? errval : PK_WARN;
02d1f3
         }
02d1f3
-        fclose(G.outfile);                  /* close "link" file for good... */
02d1f3
+        errval = CloseError(G.outfile, G.filename); /* close "link" file for good... */
02d1f3
         slnk_entry->target[ucsize] = '\0';
02d1f3
         if (QCOND2)
02d1f3
             Info(slide, 0, ((char *)slide, "-> %s ",
02d1f3
@@ -1188,7 +1220,7 @@ void close_outfile(__G)    /* GRR: chang
02d1f3
         else
02d1f3
             G.slink_head = slnk_entry;
02d1f3
         G.slink_last = slnk_entry;
02d1f3
-        return;
02d1f3
+        return errval;
02d1f3
     }
02d1f3
 #endif /* SYMLINKS */
02d1f3
 
02d1f3
@@ -1201,7 +1233,7 @@ void close_outfile(__G)    /* GRR: chang
02d1f3
 #endif
02d1f3
 
02d1f3
 #if (defined(NO_FCHOWN))
02d1f3
-    fclose(G.outfile);
02d1f3
+    errval = CloseError(G.outfile, G.filename);
02d1f3
 #endif
02d1f3
 
02d1f3
     /* if -X option was specified and we have UID/GID info, restore it */
02d1f3
@@ -1227,7 +1259,7 @@ void close_outfile(__G)    /* GRR: chang
02d1f3
     }
02d1f3
 
02d1f3
 #if (!defined(NO_FCHOWN) && defined(NO_FCHMOD))
02d1f3
-    fclose(G.outfile);
02d1f3
+    errval = CloseError(G.outfile, G.filename);
02d1f3
 #endif
02d1f3
 
02d1f3
 #if (!defined(NO_FCHOWN) && !defined(NO_FCHMOD))
02d1f3
@@ -1239,7 +1271,7 @@ void close_outfile(__G)    /* GRR: chang
02d1f3
     if (fchmod(fileno(G.outfile), filtattr(__G__ G.pInfo->file_attr)))
02d1f3
         perror("fchmod (file attributes) error");
02d1f3
 
02d1f3
-    fclose(G.outfile);
02d1f3
+    errval = CloseError(G.outfile, G.filename);
02d1f3
 #endif /* !NO_FCHOWN && !NO_FCHMOD */
02d1f3
 
02d1f3
     /* skip restoring time stamps on user's request */
02d1f3
@@ -1267,6 +1299,7 @@ void close_outfile(__G)    /* GRR: chang
02d1f3
 #endif
02d1f3
 #endif /* NO_FCHOWN || NO_FCHMOD */
02d1f3
 
02d1f3
+    return errval;
02d1f3
 } /* end function close_outfile() */
02d1f3
 
02d1f3
 #endif /* !MTS */
02d1f3
diff -up unzip60/unzpriv.h.close unzip60/unzpriv.h
02d1f3
--- unzip60/unzpriv.h.close	2009-04-20 01:59:26.000000000 +0200
02d1f3
+++ unzip60/unzpriv.h	2009-11-19 08:19:08.610388618 +0100
02d1f3
@@ -2604,7 +2604,7 @@ char    *GetLoadPath     OF((__GPRO));  
02d1f3
    int   SetFileSize     OF((FILE *file, zusz_t filesize));         /* local */
02d1f3
 #endif
02d1f3
 #ifndef MTS /* macro in MTS */
02d1f3
-   void  close_outfile   OF((__GPRO));                              /* local */
02d1f3
+   int  close_outfile   OF((__GPRO));                              /* local */
02d1f3
 #endif
02d1f3
 #ifdef SET_SYMLINK_ATTRIBS
02d1f3
    int  set_symlnk_attribs  OF((__GPRO__ slinkentry *slnk_entry));  /* local */