Commit 934fcf78 by Carlos Martín Nieto

Initialise the config reader in config_parse

git_config_open shouldn't have to initialise variables that are only
used inside config_parse and its callees.

Signed-off-by: Carlos Martín Nieto <cmn@elego.de>
parent 2974aa94
...@@ -86,9 +86,6 @@ int git_config_open(git_config **cfg_out, const char *path) ...@@ -86,9 +86,6 @@ int git_config_open(git_config **cfg_out, const char *path)
if(error < GIT_SUCCESS) if(error < GIT_SUCCESS)
goto cleanup; goto cleanup;
/* Initialise the reading position */
cfg->reader.read_ptr = cfg->reader.buffer.data;
error = config_parse(cfg); error = config_parse(cfg);
if(error < GIT_SUCCESS) if(error < GIT_SUCCESS)
git_config_free(cfg); git_config_free(cfg);
...@@ -599,6 +596,10 @@ static int config_parse(git_config *cfg_file) ...@@ -599,6 +596,10 @@ static int config_parse(git_config *cfg_file)
int error = GIT_SUCCESS; int error = GIT_SUCCESS;
char *current_section = NULL; char *current_section = NULL;
/* Initialise the reading position */
cfg_file->reader.read_ptr = cfg_file->reader.buffer.data;
cfg_file->reader.eof = 0;
skip_bom(cfg_file); skip_bom(cfg_file);
while (error == GIT_SUCCESS && !cfg_file->reader.eof) { while (error == GIT_SUCCESS && !cfg_file->reader.eof) {
......
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