On Mon, 12 Jan 2009, Bert Wesarg wrote: > On Sun, Jan 11, 2009 at 21:12, Daniel Barkalow <barkalow@xxxxxxxxxxxx> wrote: > > diff --git a/builtin-push.c b/builtin-push.c > > index 122fdcf..3fdedba 100644 > > --- a/builtin-push.c > > +++ b/builtin-push.c > > @@ -53,6 +53,9 @@ static int do_push(const char *repo, int flags) > > int i, errs; > > struct remote *remote = remote_get(repo); > > > > + if (remote->foreign_vcs) > > + die("Pushing with foreign VCSes not supported."); > > + > > if (!remote) > > die("bad repository '%s'", repo); > > > Use of remote before NULL check. Good catch, thanks. -Daniel *This .sig left intentionally blank* -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html