You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@subversion.apache.org by ju...@apache.org on 2010/08/10 12:16:39 UTC

svn commit: r983913 - /subversion/trunk/subversion/libsvn_wc/adm_ops.c

Author: julianfoad
Date: Tue Aug 10 10:16:39 2010
New Revision: 983913

URL: http://svn.apache.org/viewvc?rev=983913&view=rev
Log:
Add a couple of argument assertions.

* subversion/libsvn_wc/adm_ops.c
  (svn_wc__process_committed_internal, svn_wc_queue_committed3): Assert that
    neither checksum is given or both checksums are given.

Modified:
    subversion/trunk/subversion/libsvn_wc/adm_ops.c

Modified: subversion/trunk/subversion/libsvn_wc/adm_ops.c
URL: http://svn.apache.org/viewvc/subversion/trunk/subversion/libsvn_wc/adm_ops.c?rev=983913&r1=983912&r2=983913&view=diff
==============================================================================
--- subversion/trunk/subversion/libsvn_wc/adm_ops.c (original)
+++ subversion/trunk/subversion/libsvn_wc/adm_ops.c Tue Aug 10 10:16:39 2010
@@ -247,6 +247,11 @@ svn_wc__process_committed_internal(svn_w
 {
   svn_wc__db_kind_t kind;
 
+  SVN_ERR_ASSERT((md5_checksum == NULL && sha1_checksum == NULL)
+                 || (md5_checksum != NULL && sha1_checksum != NULL
+                     && md5_checksum->kind == svn_checksum_md5
+                     && sha1_checksum->kind == svn_checksum_sha1));
+
   SVN_ERR(svn_wc__db_read_kind(&kind, db, local_abspath, TRUE, scratch_pool));
 
   SVN_ERR(process_committed_leaf(db, local_abspath, !top_of_recurse,
@@ -411,6 +416,10 @@ svn_wc_queue_committed3(svn_wc_committed
   committed_queue_item_t *cqi;
 
   SVN_ERR_ASSERT(svn_dirent_is_absolute(local_abspath));
+  SVN_ERR_ASSERT((md5_checksum == NULL && sha1_checksum == NULL)
+                 || (md5_checksum != NULL && sha1_checksum != NULL
+                     && md5_checksum->kind == svn_checksum_md5
+                     && sha1_checksum->kind == svn_checksum_sha1));
 
   queue->have_recursive |= recurse;