Security failures are ARCHIVE_FAILED (cannot continue with this

operation), not ARCHIVE_WARN.

SVN-Revision: 135

diff --git a/libarchive/archive_write_disk.c b/libarchive/archive_write_disk.c
index 53e2f44162d5d7905aaa950789d9e48e094e4030..53c62a6c81c3bb9401ffe94ea068a626105b952c 100644 (file)
--- a/libarchive/archive_write_disk.c
+++ b/libarchive/archive_write_disk.c
@@ -1351,7 +1351,7 @@ cleanup_pathname(struct archive_write_disk *a)
        if (*src == '\0') {
                archive_set_error(&a->archive, ARCHIVE_ERRNO_MISC,
                    "Invalid empty pathname");
-               return (ARCHIVE_WARN);
+               return (ARCHIVE_FAILED);
        }
 
        /* Skip leading '/'. */
@@ -1382,7 +1382,7 @@ cleanup_pathname(struct archive_write_disk *a)
                                                archive_set_error(&a->archive,
                                                    ARCHIVE_ERRNO_MISC,
                                                    "Path contains '..'");
-                                               return (ARCHIVE_WARN);
+                                               return (ARCHIVE_FAILED);
                                        }
                                        lastdotdot = 1;
                                } else
@@ -1421,7 +1421,7 @@ cleanup_pathname(struct archive_write_disk *a)
                archive_set_error(&a->archive,
                    ARCHIVE_ERRNO_MISC,
                    "Path contains trailing '..'");
-               return (ARCHIVE_WARN);
+               return (ARCHIVE_FAILED);
        }
        if (dest == a->name) {
                /*