mirror of
https://github.com/git/git.git
synced 2024-11-18 05:13:58 +01:00
config.c:store_write_pair(): don't read the byte before a malloc'd buffer.
Signed-off-by: Jim Meyering <meyering@redhat.com> Signed-off-by: Junio C Hamano <gitster@pobox.com>
This commit is contained in:
parent
8e7425da78
commit
6281f39467
10
config.c
10
config.c
@ -630,13 +630,19 @@ static int store_write_pair(int fd, const char* key, const char* value)
|
||||
int length = strlen(key+store.baselen+1);
|
||||
int quote = 0;
|
||||
|
||||
/* Check to see if the value needs to be quoted. */
|
||||
/*
|
||||
* Check to see if the value needs to be surrounded with a dq pair.
|
||||
* Note that problematic characters are always backslash-quoted; this
|
||||
* check is about not losing leading or trailing SP and strings that
|
||||
* follow beginning-of-comment characters (i.e. ';' and '#') by the
|
||||
* configuration parser.
|
||||
*/
|
||||
if (value[0] == ' ')
|
||||
quote = 1;
|
||||
for (i = 0; value[i]; i++)
|
||||
if (value[i] == ';' || value[i] == '#')
|
||||
quote = 1;
|
||||
if (value[i-1] == ' ')
|
||||
if (i && value[i-1] == ' ')
|
||||
quote = 1;
|
||||
|
||||
if (write_in_full(fd, "\t", 1) != 1 ||
|
||||
|
Loading…
Reference in New Issue
Block a user