Commit a2366c94 by Vicent Martí

Merge pull request #467 from oleganza/oa-config-parse-fix

Fixed crash in config parser when empty value is encountered.
parents d533c888 9f861826
...@@ -1158,19 +1158,25 @@ static int parse_variable(diskfile_backend *cfg, char **var_name, char **var_val ...@@ -1158,19 +1158,25 @@ static int parse_variable(diskfile_backend *cfg, char **var_name, char **var_val
while (isspace(value_start[0])) while (isspace(value_start[0]))
value_start++; value_start++;
if (value_start[0] == '\0') if (value_start[0] == '\0') {
*var_value = NULL;
goto out; goto out;
}
if (is_multiline_var(value_start)) { if (is_multiline_var(value_start)) {
error = parse_multiline_variable(cfg, value_start, var_value); error = parse_multiline_variable(cfg, value_start, var_value);
if (error < GIT_SUCCESS) if (error != GIT_SUCCESS)
{
*var_value = NULL;
free(*var_name); free(*var_name);
}
goto out; goto out;
} }
tmp = strdup(value_start); tmp = strdup(value_start);
if (tmp == NULL) { if (tmp == NULL) {
free(*var_name); free(*var_name);
*var_value = NULL;
error = GIT_ENOMEM; error = GIT_ENOMEM;
goto out; goto out;
} }
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment