Right problem, wrong solution. Also missed an '= 1' , thanks Marcus

diff --git a/ext/phar/phar_internal.h b/ext/phar/phar_internal.h
index 7b47c3004d30ebd4b63fe4352e98b1811488f124..7cc44b4fc88ca0fdd5d1928b838979188298769b 100755 (executable)
--- a/ext/phar/phar_internal.h
+++ b/ext/phar/phar_internal.h
@@ -338,6 +338,8 @@ union _phar_entry_object {
 };
 #endif
 
+BEGIN_EXTERN_C()
+
 int phar_has_bz2;
 int phar_has_zlib;
 
@@ -357,8 +359,6 @@ static inline void phar_unixify_path_separators(char *path, int path_len)
 }
 #endif
 
-BEGIN_EXTERN_C()
-
 void phar_request_initialize(TSRMLS_D);
 
 void phar_object_init(TSRMLS_D);

diff --git a/ext/phar/zip.c b/ext/phar/zip.c
index f76b7b0b83fd1f26ba301336919b0ffe5dbfcf7b..9beffedf63ac53b6f9933d00f644cdab3d5425d1 100644 (file)
--- a/ext/phar/zip.c
+++ b/ext/phar/zip.c
@@ -386,7 +386,7 @@ int phar_open_or_create_zip(char *fname, int fname_len, char *alias, int alias_l
        phar->is_data = is_data;
 
        if (is_data) {
-               phar->is_writeable;
+               phar->is_writeable = 1;
                phar->alias = NULL;
                phar->alias_len = 0;
        }