diff options
author | Jeff King <peff@peff.net> | 2017-04-24 20:57:47 -0400 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2017-04-24 19:00:15 -0700 |
commit | 627fde102515a7807dba89acaa88cb053b38a44a (patch) | |
tree | 3a255ca92f07db454e523baff730b18941eae6a8 /parse-options.h | |
parent | Prepare for 2.9.4 (diff) | |
download | tgif-627fde102515a7807dba89acaa88cb053b38a44a.tar.xz |
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 'parse-options.h')
0 files changed, 0 insertions, 0 deletions