kernel
/
git
mirror of https://git.kernel.org/pub/scm/git/git.git
1
0
Fork 0
Code Releases Activity

git-svn: check error code of send_txstream 

Not checking the error code of a function used to transform and
send data makes me nervous.  It currently returns "undef" on
success; so die if we get any result other than "undef" because
it's likely something went wrong somewhere.  I really wish this
function returned an MD5 like send_stream (or better yet, SHA1)
for verification.

Signed-off-by: Eric Wong <normalperson@yhbt.net>
maint
Eric Wong 2008-08-31 19:45:07 -07:00
parent 8598db935b
commit 991255c634
1 changed files with 5 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
git-svn.perl
View File

@ -3697,7 +3697,11 @@ sub chg_file {
my $atd = $self->apply_textdelta($fbat, $exp_a, $pool); my $atd = $self->apply_textdelta($fbat, $exp_a, $pool);
if (-s $fh_a) { if (-s $fh_a) {
my $txstream = SVN::TxDelta::new ($fh_a, $fh_b, $pool); my $txstream = SVN::TxDelta::new ($fh_a, $fh_b, $pool);
SVN::TxDelta::send_txstream($txstream, @$atd, $pool); my $res = SVN::TxDelta::send_txstream($txstream, @$atd, $pool);
if (defined $res) {
die "Unexpected result from send_txstream: $res\n",
"(SVN::Core::VERSION: $SVN::Core::VERSION)\n";
}
} else { } else {
my $got = SVN::TxDelta::send_stream($fh_b, @$atd, $pool); my $got = SVN::TxDelta::send_stream($fh_b, @$atd, $pool);
die "Checksum mismatch\nexpected: $exp_b\ngot: $got\n" die "Checksum mismatch\nexpected: $exp_b\ngot: $got\n"