diff options
author | Jeff King <peff@peff.net> | 2017-04-25 00:57:47 (GMT) |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2017-04-25 02:00:15 (GMT) |
commit | 627fde102515a7807dba89acaa88cb053b38a44a (patch) | |
tree | 3a255ca92f07db454e523baff730b18941eae6a8 /builtin/pack-refs.c | |
parent | 0202c411edc25940cc381bf317badcdf67670be4 (diff) | |
download | git-627fde102515a7807dba89acaa88cb053b38a44a.zip git-627fde102515a7807dba89acaa88cb053b38a44a.tar.gz git-627fde102515a7807dba89acaa88cb053b38a44a.tar.bz2 |
submodule_init: die cleanly on submodules without url defined
When we init a submodule, we try to die when it has no URL
defined:
url = xstrdup(sub->url);
if (!url)
die(...);
But that's clearly nonsense. xstrdup() will never return
NULL, and if sub->url is NULL, we'll segfault.
These two bits of code need to be flipped, so we check
sub->url before looking at it.
Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'builtin/pack-refs.c')
0 files changed, 0 insertions, 0 deletions