Use ntohl instead of doing LE conversion yourself

[m6w6/ext-http] / src / php_http_etag.c

diff --git a/src/php_http_etag.c b/src/php_http_etag.c
index a02548cc028c5d0c81103abc09ce029a60588353..1430cfa1c6726dec7b6b2f3264fa4cf0abf0012c 100644 (file)
--- a/src/php_http_etag.c
+++ b/src/php_http_etag.c
@@ -12,8 +12,8 @@
 
 #include "php_http_api.h"
 
-#ifdef PHP_HTTP_HAVE_HASH
-#      include "php_hash.h"
+#if PHP_HTTP_HAVE_HASH
+#      include "ext/hash/php_hash.h"
 #endif
 
 #include "ext/standard/crc32.h"
@@ -33,7 +33,7 @@ php_http_etag_t *php_http_etag_init(const char *mode)
        } else if (mode && !strcasecmp(mode, "md5")) {
                PHP_MD5Init(ctx = emalloc(sizeof(PHP_MD5_CTX)));
        } else {
-#ifdef PHP_HTTP_HAVE_HASH
+#if PHP_HTTP_HAVE_HASH
                const php_hash_ops *eho = NULL;
 
                if (mode && (eho = php_hash_fetch_ops(mode, strlen(mode)))) {
@@ -57,18 +57,10 @@ char *php_http_etag_finish(php_http_etag_t *e)
        char *etag = NULL;
 
        if (!strcasecmp(e->mode, "crc32b")) {
-               unsigned char buf[4];
-
-               *((uint *) e->ctx) = ~*((uint *) e->ctx);
-#ifdef WORDS_BIGENDIAN
-               etag = php_http_etag_digest((unsigned char *) e->ctx, 4);
-#else
-               buf[0] = ((unsigned char *) e->ctx)[3];
-               buf[1] = ((unsigned char *) e->ctx)[2];
-               buf[2] = ((unsigned char *) e->ctx)[1];
-               buf[3] = ((unsigned char *) e->ctx)[0];
-               etag = php_http_etag_digest(buf, 4);
-#endif
+               uint e_ctx;
+               memcpy(&e_ctx, e->ctx, 4);
+               e_ctx = ntohl(~e_ctx);
+               etag = php_http_etag_digest((unsigned char *) &e_ctx, 4);
        } else if ((!strcasecmp(e->mode, "sha1"))) {
                PHP_SHA1Final(digest, e->ctx);
                etag = php_http_etag_digest(digest, 20);
@@ -76,7 +68,7 @@ char *php_http_etag_finish(php_http_etag_t *e)
                PHP_MD5Final(digest, e->ctx);
                etag = php_http_etag_digest(digest, 16);
        } else {
-#ifdef PHP_HTTP_HAVE_HASH
+#if PHP_HTTP_HAVE_HASH
                const php_hash_ops *eho = NULL;
 
                if ((eho = php_hash_fetch_ops(e->mode, strlen(e->mode)))) {
@@ -106,7 +98,7 @@ size_t php_http_etag_update(php_http_etag_t *e, const char *data_ptr, size_t dat
        } else if ((!strcasecmp(e->mode, "md5"))) {
                PHP_MD5Update(e->ctx, (const unsigned char *) data_ptr, data_len);
        } else {
-#ifdef PHP_HTTP_HAVE_HASH
+#if PHP_HTTP_HAVE_HASH
                const php_hash_ops *eho = NULL;
 
                if ((eho = php_hash_fetch_ops(e->mode, strlen(e->mode)))) {