]> granicus.if.org Git - pgbouncer/commitdiff
Avoid NULL pointer deref in parse_line
authorDaniel Gustafsson <dgustafsson@pivotal.io>
Thu, 7 Jun 2018 21:04:29 +0000 (23:04 +0200)
committerDaniel Gustafsson <dgustafsson@pivotal.io>
Thu, 7 Jun 2018 21:04:29 +0000 (23:04 +0200)
If we fail to calloc the rule buffer, going to failed: will deref
rule via the rule_free() call. Avoid by returning false directly
and bypass the failed: block.

src/hba.c

index ca29ea3bcf79c81515de504c558f175c69f34333..100edf6ea9f196c0ab3c7256bf8ede5b3159481f 100644 (file)
--- a/src/hba.c
+++ b/src/hba.c
@@ -524,7 +524,7 @@ static bool parse_line(struct HBA *hba, struct TokParser *tp, int linenr, const
        rule = calloc(sizeof *rule, 1);
        if (!rule) {
                log_warning("hba: no mem for rule");
-               goto failed;
+               return false;
        }
        rule->rule_type = rtype;