Prevent stale stub functions from being called in deflate_slow

Just in case this is the very first call to longest match, we should
instead assign the function pointer instead of the function itself. This
way, by the time it leaves the stub, the function pointer gets
reassigned. This was found incidentally while debugging something else.

diff --git a/deflate_slow.c b/deflate_slow.c
index b8beec9ccd28d2f7c628d5e9766aeadd1c6956d9..20fa0f39ac7283660363dc19a7f627550a2e0847 100644 (file)
--- a/deflate_slow.c
+++ b/deflate_slow.c
@@ -19,12 +19,12 @@ Z_INTERNAL block_state deflate_slow(deflate_state *s, int flush) {
     int bflush;              /* set if current block must be flushed */
     int64_t dist;
     uint32_t match_len;
-    match_func longest_match;
+    match_func *longest_match;
 
     if (s->max_chain_length <= 1024)
-        longest_match = functable.longest_match;
+        longest_match = &functable.longest_match;
     else
-        longest_match = functable.longest_match_slow;
+        longest_match = &functable.longest_match_slow;
 
     /* Process the input block. */
     for (;;) {
@@ -61,7 +61,7 @@ Z_INTERNAL block_state deflate_slow(deflate_state *s, int flush) {
              * of window index 0 (in particular we have to avoid a match
              * of the string with itself at the start of the input file).
              */
-            match_len = longest_match(s, hash_head);
+            match_len = (*longest_match)(s, hash_head);
             /* longest_match() sets match_start */
 
             if (match_len <= 5 && (s->strategy == Z_FILTERED)) {