Fix use after free. Bug found by Coverity

diff --git a/src/lib/valuepair.c b/src/lib/valuepair.c
index 965d107e599d0beebb1f4cdf27e00d42ef2e2d60..e5f2d9d3ad1781049122409146a0250f0b17d633 100644 (file)
--- a/src/lib/valuepair.c
+++ b/src/lib/valuepair.c
@@ -958,8 +958,8 @@ VALUE_PAIR *pairparsevalue(VALUE_PAIR *vp, const char *value)
                                fr_ipaddr_t ipaddr;
 
                                if (ip_hton(cs, AF_INET, &ipaddr) < 0) {
-                                       free(s);
                                        fr_strerror_printf("Failed to find IP address for %s", cs);
+                                       free(s);
                                        return NULL;
                                }