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);