You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@subversion.apache.org by ko...@apache.org on 2017/08/10 14:01:34 UTC
svn commit: r1804672 - /subversion/trunk/subversion/libsvn_client/import.c
Author: kotkov
Date: Thu Aug 10 14:01:34 2017
New Revision: 1804672
URL: http://svn.apache.org/viewvc?rev=1804672&view=rev
Log:
Following up on r1803144, fix an oversight in the txdelta_next_window
adapter used when importing files over HTTP.
This bug has inadvertently disabled the checksum validation for such
imports, because the resulting checksum has always been all-zeroes.
* subversion/libsvn_client/import.c
(txdelta_next_window): Check if the svn_txdelta_window_t pointer
is null, instead of checking that for the pointer to a pointer, which
is never null.
Modified:
subversion/trunk/subversion/libsvn_client/import.c
Modified: subversion/trunk/subversion/libsvn_client/import.c
URL: http://svn.apache.org/viewvc/subversion/trunk/subversion/libsvn_client/import.c?rev=1804672&r1=1804671&r2=1804672&view=diff
==============================================================================
--- subversion/trunk/subversion/libsvn_client/import.c (original)
+++ subversion/trunk/subversion/libsvn_client/import.c Thu Aug 10 14:01:34 2017
@@ -87,7 +87,7 @@ txdelta_next_window(svn_txdelta_window_t
struct txdelta_stream_baton_t *b = baton;
SVN_ERR(svn_txdelta_next_window(window, b->txstream, pool));
- if (!window)
+ if (!*window)
{
memcpy(b->digest_buffer, svn_txdelta_md5_digest(b->txstream),
APR_MD5_DIGESTSIZE);