[PATCH 02/13] BUG/MINOR: deviceatlas: add NULL checks on strdup() results in config parsers

[David Carlier]

From: David Carlier <dcarl...@deviceatlas.com>

Add missing NULL checks after strdup() for the json file path in
da_json_file() and the cookie name in da_properties_cookie().

This should be backported to lower branches.
---
 addons/deviceatlas/da.c | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/addons/deviceatlas/da.c b/addons/deviceatlas/da.c
index e9fb91d1a..3b1576678 100644
--- a/addons/deviceatlas/da.c
+++ b/addons/deviceatlas/da.c
@@ -57,6 +57,10 @@ static int da_json_file(char **args, int section_type, 
struct proxy *curpx,
                return -1;
        }
        global_deviceatlas.jsonpath = strdup(args[1]);
+       if (unlikely(global_deviceatlas.jsonpath == NULL)) {
+               memprintf(err, "deviceatlas json file : out of memory.\n");
+               return -1;
+       }
        return 0;
 }
 
@@ -102,6 +106,10 @@ static int da_properties_cookie(char **args, int 
section_type, struct proxy *cur
                return -1;
        } else {
                global_deviceatlas.cookiename = strdup(args[1]);
+               if (unlikely(global_deviceatlas.cookiename == NULL)) {
+                       memprintf(err, "deviceatlas cookie name : out of 
memory.\n");
+                       return -1;
+               }
        }
        global_deviceatlas.cookienamelen = 
strlen(global_deviceatlas.cookiename);
        return 0;
-- 
2.51.0