MINOR: http: check the algo name "identity" instead of the function pointer
Next patch will statity all compression functions, so let's stop relying
on a function pointer comparison and use the algo name instead.
diff --git a/src/proto_http.c b/src/proto_http.c
index 83dd3a4..56e4d3f 100644
--- a/src/proto_http.c
+++ b/src/proto_http.c
@@ -2333,7 +2333,7 @@
/* identity is implicit does not require headers */
if ((s->be->comp && (comp_algo_back = s->be->comp->algos)) || (s->fe->comp && (comp_algo_back = s->fe->comp->algos))) {
for (comp_algo = comp_algo_back; comp_algo; comp_algo = comp_algo->next) {
- if (comp_algo->add_data == identity_add_data) {
+ if (comp_algo->name_len == 8 && memcmp(comp_algo->name, "identity", 8) == 0) {
s->comp_algo = comp_algo;
return 1;
}
@@ -2445,7 +2445,7 @@
* Accept-Encoding header, and SHOULD NOT be used in the Content-Encoding
* header.
*/
- if (s->comp_algo->add_data != identity_add_data) {
+ if (s->comp_algo->name_len != 8 || memcmp(s->comp_algo->name, "identity", 8) != 0) {
trash.len = 18;
memcpy(trash.str, "Content-Encoding: ", trash.len);
memcpy(trash.str + trash.len, s->comp_algo->name, s->comp_algo->name_len);