Skip to content

Commit

Permalink
receive-pack: check connectivity before concluding "git push"
Browse files Browse the repository at this point in the history
Signed-off-by: Junio C Hamano <gitster@pobox.com>
  • Loading branch information
Junio C Hamano committed Sep 9, 2011
1 parent f96400c commit 52fed6e
Showing 1 changed file with 43 additions and 0 deletions.
43 changes: 43 additions & 0 deletions builtin/receive-pack.c
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,7 @@
#include "transport.h"
#include "string-list.h"
#include "sha1-array.h"
#include "connected.h"

static const char receive_pack_usage[] = "git receive-pack <git-dir>";

Expand Down Expand Up @@ -546,6 +547,43 @@ static void check_aliased_updates(struct command *commands)
string_list_clear(&ref_list, 0);
}

static int command_singleton_iterator(void *cb_data, unsigned char sha1[20])
{
struct command **cmd_list = cb_data;
struct command *cmd = *cmd_list;

if (!cmd)
return -1; /* end of list */
*cmd_list = NULL; /* this returns only one */
hashcpy(sha1, cmd->new_sha1);
return 0;
}

static void set_connectivity_errors(struct command *commands)
{
struct command *cmd;

for (cmd = commands; cmd; cmd = cmd->next) {
struct command *singleton = cmd;
if (!check_everything_connected(command_singleton_iterator,
0, &singleton))
continue;
cmd->error_string = "missing necessary objects";
}
}

static int iterate_receive_command_list(void *cb_data, unsigned char sha1[20])
{
struct command **cmd_list = cb_data;
struct command *cmd = *cmd_list;

if (!cmd)
return -1; /* end of list */
*cmd_list = cmd->next;
hashcpy(sha1, cmd->new_sha1);
return 0;
}

static void execute_commands(struct command *commands, const char *unpacker_error)
{
struct command *cmd;
Expand All @@ -557,6 +595,11 @@ static void execute_commands(struct command *commands, const char *unpacker_erro
return;
}

cmd = commands;
if (check_everything_connected(iterate_receive_command_list,
0, &cmd))
set_connectivity_errors(commands);

if (run_receive_hook(commands, pre_receive_hook)) {
for (cmd = commands; cmd; cmd = cmd->next)
cmd->error_string = "pre-receive hook declined";
Expand Down

0 comments on commit 52fed6e

Please sign in to comment.