You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@subversion.apache.org by br...@apache.org on 2013/09/04 14:37:40 UTC
svn commit: r1519995 -
/subversion/trunk/subversion/bindings/javahl/native/RemoteSession.cpp
Author: brane
Date: Wed Sep 4 12:37:39 2013
New Revision: 1519995
URL: http://svn.apache.org/r1519995
Log:
Try not to assign values through NULL pointers.
* subversion/bindings/javahl/native/RemoteSession.cpp
(FileRevisionHandler::callback): Check the delta_handler and
delta_handler_baton pointers before assigning values to the
variables they refer to.
Modified:
subversion/trunk/subversion/bindings/javahl/native/RemoteSession.cpp
Modified: subversion/trunk/subversion/bindings/javahl/native/RemoteSession.cpp
URL: http://svn.apache.org/viewvc/subversion/trunk/subversion/bindings/javahl/native/RemoteSession.cpp?rev=1519995&r1=1519994&r2=1519995&view=diff
==============================================================================
--- subversion/trunk/subversion/bindings/javahl/native/RemoteSession.cpp (original)
+++ subversion/trunk/subversion/bindings/javahl/native/RemoteSession.cpp Wed Sep 4 12:37:39 2013
@@ -1177,8 +1177,10 @@ public:
apr_array_header_t* prop_diffs,
apr_pool_t* scratch_pool)
{
- *delta_handler = svn_delta_noop_window_handler;
- *delta_handler_baton = NULL;
+ if (delta_handler)
+ *delta_handler = svn_delta_noop_window_handler;
+ if (delta_handler_baton)
+ *delta_handler_baton = NULL;
FileRevisionHandler* const self =
static_cast<FileRevisionHandler*>(baton);