You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@subversion.apache.org by st...@apache.org on 2015/09/10 19:18:10 UTC

svn commit: r1702302 - /subversion/trunk/subversion/libsvn_subr/stream.c

Author: stefan2
Date: Thu Sep 10 17:18:10 2015
New Revision: 1702302

URL: http://svn.apache.org/r1702302
Log:
Don't make svn_stream_skip segfault on write-only streams.

* subversion/libsvn_subr/stream.c
  (svn_stream_skip): The fallback only applies when we can read data at all.

Modified:
    subversion/trunk/subversion/libsvn_subr/stream.c

Modified: subversion/trunk/subversion/libsvn_subr/stream.c
URL: http://svn.apache.org/viewvc/subversion/trunk/subversion/libsvn_subr/stream.c?rev=1702302&r1=1702301&r2=1702302&view=diff
==============================================================================
--- subversion/trunk/subversion/libsvn_subr/stream.c (original)
+++ subversion/trunk/subversion/libsvn_subr/stream.c Thu Sep 10 17:18:10 2015
@@ -200,6 +200,9 @@ svn_stream_skip(svn_stream_t *stream, ap
     {
       svn_read_fn_t read_fn = stream->read_full_fn ? stream->read_full_fn
                                                    : stream->read_fn;
+      if (read_fn == NULL)
+        return svn_error_create(SVN_ERR_STREAM_NOT_SUPPORTED, NULL, NULL);
+
       return svn_error_trace(skip_default_handler(stream->baton, len,
                                                   read_fn));
     }