- print doesn't like commas

[m6w6/ext-http] / php_http_cache_api.h

diff --git a/php_http_cache_api.h b/php_http_cache_api.h
index bef8cc8b97d14cdf94548db85c82868e261e3ddb..4f067f5a0c72c5fcb8936faa3e80f3f3a52b95f8 100644 (file)
--- a/php_http_cache_api.h
+++ b/php_http_cache_api.h
@@ -28,7 +28,7 @@
 #include "php_http_api.h"
 #include "php_http_send_api.h"
 
-#ifdef HAVE_LIBMHASH
+#ifdef HTTP_HAVE_MHASH
 #      include <mhash.h>
 #endif
 
@@ -40,7 +40,7 @@ typedef enum {
        HTTP_ETAG_MHASH = 0,
 } http_etag_mode;
 
-#ifdef HAVE_LIBMHASH
+#ifdef HTTP_HAVE_MHASH
 static void *http_etag_alloc_mhash_digest(size_t size)
 {
        return emalloc(size);
@@ -68,7 +68,7 @@ static inline char *_http_etag_digest(const unsigned char *digest, int len TSRML
 #define http_etag_init() _http_etag_init(TSRMLS_C)
 static inline void *_http_etag_init(TSRMLS_D)
 {
-       void *ctx;
+       void *ctx = NULL;
        long mode = INI_INT("http.etag_mode");
        
        switch (mode)
@@ -78,24 +78,19 @@ static inline void *_http_etag_init(TSRMLS_D)
                break;
                
                case HTTP_ETAG_MD5:
-invalid_flag:
+#ifndef HTTP_HAVE_MHASH
+               default:
+#endif
                        PHP_MD5Init(ctx = emalloc(sizeof(PHP_MD5_CTX)));
                break;
                
+#ifdef HTTP_HAVE_MHASH
                default:
-               {
-#ifdef HAVE_LIBMHASH
-                       if ((mode >= 0) && (mode <= mhash_count())) {
-                               ctx = mhash_init(mode);
-                       }
-                       if ((!ctx) || (ctx == MHASH_FAILED))
-#endif
-                       {
-                               HTTP_G(etag).mode = HTTP_ETAG_MD5;
-                               goto invalid_flag;
+                       if ((mode < 0) || ((ulong)mode > mhash_count()) || (!(ctx = mhash_init(mode)))) {
+                               http_error_ex(HE_ERROR, HTTP_E_RUNTIME, "Invalid ETag mode: %ld", mode);
                        }
-               }
                break;
+#endif
        }
        
        return ctx;
@@ -117,20 +112,23 @@ static inline char *_http_etag_finish(void *ctx TSRMLS_DC)
                break;
                
                case HTTP_ETAG_MD5:
+#ifndef HTTP_HAVE_MHASH
+               default:
+#endif
                        PHP_MD5Final(digest, ctx);
                        etag = http_etag_digest(digest, 16);
                        efree(ctx);
                break;
                
+#ifdef HTTP_HAVE_MHASH
                default:
                {
-#ifdef HAVE_LIBMHASH
                        unsigned char *mhash_digest = mhash_end_m(ctx, http_etag_alloc_mhash_digest);
                        etag = http_etag_digest(mhash_digest, mhash_get_block_size(mode));
                        efree(mhash_digest);
-#endif
                }
                break;
+#endif
        }
        
        return etag;
@@ -142,18 +140,21 @@ static inline void _http_etag_update(void *ctx, const char *data_ptr, size_t dat
        switch (INI_INT("http.etag_mode"))
        {
                case HTTP_ETAG_SHA1:
-                       PHP_SHA1Update(ctx, data_ptr, data_len);
+                       PHP_SHA1Update(ctx, (const unsigned char *) data_ptr, data_len);
                break;
                
                case HTTP_ETAG_MD5:
-                       PHP_MD5Update(ctx, data_ptr, data_len);
+#ifndef HTTP_HAVE_MHASH
+               default:
+#endif
+                       PHP_MD5Update(ctx, (const unsigned char *) data_ptr, data_len);
                break;
                
+#ifdef HTTP_HAVE_MHASH
                default:
-#ifdef HAVE_LIBMHASH
                        mhash(ctx, data_ptr, data_len);
-#endif
                break;
+#endif
        }
 }