diff options
author | Junio C Hamano <gitster@pobox.com> | 2015-03-06 00:13:00 +0300 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2015-03-06 00:13:00 +0300 |
commit | 2250406bfdf8d9d890f53b651868363a304a7701 (patch) | |
tree | f3b6347563755f13f9d4af9f62d08dae8654c1f1 /config.c | |
parent | 3bef3c12d6c0f5be0e95eb143fdd32583bae7a74 (diff) | |
parent | 1d0655c15ebf7dfb460466d058daab790ed285b2 (diff) |
Merge branch 'jk/config-no-ungetc-eof' into maint
Reading configuration from a blob object, when it ends with a lone
CR, use to confuse the configuration parser.
* jk/config-no-ungetc-eof:
config_buf_ungetc: warn when pushing back a random character
config: do not ungetc EOF
Diffstat (limited to 'config.c')
-rw-r--r-- | config.c | 11 |
1 files changed, 8 insertions, 3 deletions
@@ -73,8 +73,12 @@ static int config_buf_fgetc(struct config_source *conf) static int config_buf_ungetc(int c, struct config_source *conf) { - if (conf->u.buf.pos > 0) - return conf->u.buf.buf[--conf->u.buf.pos]; + if (conf->u.buf.pos > 0) { + conf->u.buf.pos--; + if (conf->u.buf.buf[conf->u.buf.pos] != c) + die("BUG: config_buf can only ungetc the same character"); + return c; + } return EOF; } @@ -235,7 +239,8 @@ static int get_next_char(void) /* DOS like systems */ c = cf->do_fgetc(cf); if (c != '\n') { - cf->do_ungetc(c, cf); + if (c != EOF) + cf->do_ungetc(c, cf); c = '\r'; } } |