check for allocation failures

	Patch from Steve G

diff --git a/src/modules/rlm_exec/rlm_exec.c b/src/modules/rlm_exec/rlm_exec.c
index 4574381..897ae85 100644 (file)
--- a/src/modules/rlm_exec/rlm_exec.c
+++ b/src/modules/rlm_exec/rlm_exec.c
@@ -196,6 +196,8 @@ static int exec_instantiate(CONF_SECTION *conf, void **instance)
         */
        
        inst = rad_malloc(sizeof(rlm_exec_t));
+       if (!inst)
+               return -1;
        memset(inst, 0, sizeof(rlm_exec_t));
                
        /*

diff --git a/src/modules/rlm_expr/rlm_expr.c b/src/modules/rlm_expr/rlm_expr.c
index d7af020..f0d1a36 100644 (file)
--- a/src/modules/rlm_expr/rlm_expr.c
+++ b/src/modules/rlm_expr/rlm_expr.c
@@ -269,6 +269,8 @@ static int expr_instantiate(CONF_SECTION *conf, void **instance)
         */
        
        inst = rad_malloc(sizeof(rlm_expr_t));
+       if (!inst)
+               return -1;
        memset(inst, 0, sizeof(rlm_expr_t));
                
        xlat_name = cf_section_name2(conf);

diff --git a/src/modules/rlm_fastusers/rlm_fastusers.c b/src/modules/rlm_fastusers/rlm_fastusers.c
index 5420546..8d215ec 100644 (file)
--- a/src/modules/rlm_fastusers/rlm_fastusers.c
+++ b/src/modules/rlm_fastusers/rlm_fastusers.c
@@ -544,7 +544,8 @@ static int fastuser_instantiate(CONF_SECTION *conf, void **instance)
        struct fastuser_instance *inst=0;
 
        inst = rad_malloc(sizeof *inst);
-
+       if (!inst)
+               return -1;
        memset(inst, 0, sizeof(*inst));
 
        if (cf_section_parse(conf, inst, module_config) < 0) {

diff --git a/src/modules/rlm_x99_token/x99_rlm.c b/src/modules/rlm_x99_token/x99_rlm.c
index 2097b09..4bdfda4 100644 (file)
--- a/src/modules/rlm_x99_token/x99_rlm.c
+++ b/src/modules/rlm_x99_token/x99_rlm.c
@@ -143,6 +143,9 @@ x99_token_instantiate(CONF_SECTION *conf, void **instance)
 
     /* Set up a storage area for instance data. */
     data = rad_malloc(sizeof(*data));
+    if (!data)
+       return -1;
+    memset(data, 0, sizeof(*data));
 
     /* If the configuration parameters can't be parsed, then fail. */
     if (cf_section_parse(conf, data, module_config) < 0) {