BUG/MINOR: backend: fix improper insert in avail tree for always reuse

In connect_server(), if http-reuse always is set, the backend connection
is inserted into the available tree as soon as created. However, the
hash connection field is only set later at the end of the function.

This seems to have no impact as the hash connection field is always
position before a lookup. However, this is not a proper usage of ebmb
API. Fix this by setting the hash connection field before the insertion
into the avail tree.

This must be backported up to 2.4.

diff --git a/src/backend.c b/src/backend.c
index 4c7aba1f890df7f92bf604aa29794a37dc21b82b..6d77864db90228c7d6f8ccc1608726dd0af00faa 100644 (file)
--- a/src/backend.c
+++ b/src/backend.c
@@ -1523,6 +1523,8 @@ skip_reuse:
                                conn_free(srv_conn);
                                return SF_ERR_RESOURCE;
                        }
+
+                       srv_conn->hash_node->hash = hash;
                }
        }
 
@@ -1746,8 +1748,6 @@ skip_reuse:
                }
        }
 
-       srv_conn->hash_node->hash = hash;
-
        return SF_ERR_NONE;  /* connection is OK */
 }