Coverity <https://scan.coverity.com>


Found
r1738294, r1738262, r1738259, r1735916, r1735880, r877001, r872403, r872402, r872400, r863010, r859813, r859687, r859653, r859645, r859644, r859596, r859594, r859591, r859590, r859589, r859588, r859587, r859586, r859578, r859577

r1738294 | ivan | 2016-04-08 14:58:36 -0500 (Fri, 08 Apr 2016)

* subversion/libsvn_client/diff_local.c
  (svn_client__arbitrary_nodes_diff): Remove duplicate condition.

Found by: Coverity <https://scan.coverity.com>
(CID: 1331995)


r1738262 | jamessan | 2016-04-08 10:53:02 -0500 (Fri, 08 Apr 2016)

* subversion/libsvn_client/diff_local.c
  (inner_dir_diff): Set left_only, not right_only, when right_abspath doesn't exist.

Found by: Coverity <https://scan.coverity.com>
(CID: 1332090)


r1738259 | jamessan | 2016-04-08 10:28:27 -0500 (Fri, 08 Apr 2016)

* subversion/libsvn_repos/log.c
  (do_logs): Use sizeof(*cur_rev) instead of sizeof(cur_rev) to fix out of
    bounds read when sizeof(svn_revnum_t) < sizeof(svn_revnum_t *)

Found by: Coverity <https://scan.coverity.com>
(CID: 1331600)


r1735916 | jamessan | 2016-03-20 20:33:59 -0500 (Sun, 20 Mar 2016)

* subversion/libsvn_fs_fs/cached_data.c
  (skip_contents): Remove use of subpool variable in its initialization.

Found by: Coverity <https://scan.coverity.com>
(CID: 1331979)



r1735880 | jamessan | 2016-03-20 08:35:09 -0500 (Sun, 20 Mar 2016)

* subversion/tests/libsvn_diff/parse-diff-test.c
  (test_parse_git_tree_and_text_diff, test_parse_git_diff): Assert on
  {old,new}_{executable,symlink}_bit equality with expected value instead of
  the reslut of the assignment of the expected value.

Found by: Coverity <https://scan.coverity.com/>
(CID: 1356909-1356916, 1356946-1356952)



r877001 | hwright | 2009-04-01 14:48:18 -0500 (Wed, 01 Apr 2009)

Remove some dead code.

* subversion/libsvn_client/externals.c
  (switch_file_external): We don't generate any errors between this check and
    the one before it, so don't bother checking the error again.

Found by: Coverity <http://scan.coverity.com/>
(CID: 104)


r872403 | hwright | 2008-07-29 15:40:30 -0500 (Tue, 29 Jul 2008)

Fix an uninitialized variable error.

Found by: Coverity <http://scan.coverity.com/>
(CID: 77)

* subversion/libsvn_client/url.c
  (svn_client__derive_location): Initialize the cancelation function and baton
    for the case where they aren't user provided.


r872402 | hwright | 2008-07-29 15:33:47 -0500 (Tue, 29 Jul 2008)

Avoid passing a NULL value to strcmp().  Although most modern implementations
of strcmp() include a NULL check, the C89 standard, which we claim to support,
does not allow this.  A cursory web search reveals some implementations which
do crash on NULL input, so this is a valid issue.

Found by: Coverity <http://scan.coverity.com/>
(CID: 85)

* subversion/libsvn_subr/simple_providers.c
  (svn_auth__simple_save_creds_helper): Check a value for NULL before using
    it for comparison.


r872400 | hwright | 2008-07-29 14:53:06 -0500 (Tue, 29 Jul 2008)

Avoid a NULL-pointer dereference.

Found by: Coverity <http://scan.coverity.com/>
(CID: 75)

* subversion/libsvn_subr/mergeinfo.c
  (combine_with_lastrange):  Allocate a stucture before attempting to
    access its members.


r863010 | ehu | 2007-01-08 17:44:35 -0600 (Mon, 08 Jan 2007)

r22727 followup

* subversion/libsvn_wc/adm_crawler.c
  (svn_wc_transmit_text_deltas2): Remove dead code and comment cruft.

Found by: Coverity <http://scan.coverity.com/>
(CID: 47)


r859813 | malcolm | 2006-05-22 08:55:50 -0500 (Mon, 22 May 2006)

* subversion/libsvn_repos/reporter.c
  (delta_proplists): Remove redundant check for uuid being null.

See also r19570, which fixed an identical instance of this problem in delta.c.

Found by: Coverity <http://scan.coverity.com/>
(CID: 8)


r859687 | maxb | 2006-05-11 07:10:34 -0500 (Thu, 11 May 2006)

Remove some redundant NULL-ness checks, and clarify others by checking
variables more directly related to the flow of control.

Found by: Coverity <http://scan.coverity.com>
(CID: 13, 14, 15, 16)

* subversion/libsvn_subr/opt.c (svn_opt_print_help, svn_opt_print_help2):


r859653 | lundblad | 2006-05-09 10:25:52 -0500 (Tue, 09 May 2006)

* subversion/libsvn_subr/prompt.c
  (svn_cmdline_auth_ssl_server_trust_prompt): Remove redundant NULL
  pointer check.

Found by: Coverity <http://scan.coverity.com>
(CID: 26)


r859645 | lundblad | 2006-05-09 04:31:34 -0500 (Tue, 09 May 2006)

* subversion/libsvn_client/repos_diff.c
  (open_root): Remove redundant check for the target being NULL.

Found by: Coverity <http://scan.coverity.com>
(CID: 22)


r859644 | lundblad | 2006-05-09 04:10:06 -0500 (Tue, 09 May 2006)

* subversion/libsvn_repos/delta.c
  (delta_proplists): Remove redundant check for uuid being null.

Found by: Coverity <http://scan.coverity.com/>
(CID: 8)


r859596 | lundblad | 2006-05-05 02:44:50 -0500 (Fri, 05 May 2006)


* subversion/libsvn_wc/update_editor.c
  (change_file_prop): Don't crash if the last-change-date property is
  NULL.

Found by: Coverity <http://scan.coverity.com>
(CID: 6)


r859594 | lundblad | 2006-05-05 01:37:34 -0500 (Fri, 05 May 2006)

* subversion/mod_dav_svn/version.c
  (dav_svn__build_lock_hash): Don't crash if there were elements in our
  namespace, but still no lock-token-list element.
  (This doesn't happen with our current client.)

Found by: Coverity <http://scan.coverity.com>
(CID: 18)


r859591 | djames | 2006-05-04 17:02:34 -0500 (Thu, 04 May 2006)

* subversion/libsvn_ra_dav/replay.c
  (start_element): Don't push an undefined dir_baton onto the stack when
  the root element is missing a required attribute.

Found by: Coverity <http://scan.coverity.com>
(CID: 27)


r859590 | lundblad | 2006-05-04 16:18:53 -0500 (Thu, 04 May 2006)

* subversion/mod_dav_svn/lock.c
  (dav_svn_find_lock): Create a new error in a case where we were trying to
  convert a non-existing svn error.

Found by: Coverity <http://scan.coverity.com>
(CID: 19)


r859589 | djames | 2006-05-04 16:04:25 -0500 (Thu, 04 May 2006)

* subversion/libsvn_fs_base/util/fs_skels.c
  (is_valid_rep_delta_chunk_skel): Update length check to account for the fact
  that the window checksum skel was removed in r7643.

Found by: Coverity <http://scan.coverity.com>
(CID: 1)


r859588 | lundblad | 2006-05-04 15:56:34 -0500 (Thu, 04 May 2006)


* subversion/libsvn_wc/status.c
  (make_file_baton): Don't check for the parent baton being null, because it
  can't be.

Found by: Coverity <http://www.coverity.com>
(CID: 21)


r859587 | djames | 2006-05-04 15:38:24 -0500 (Thu, 04 May 2006)

* subversion/libsvn_client/copy.c
  (wc_to_repos_copy): Remove unnecessary "commit_in_progress" variable and
  associated checks. We never abort a commit in progress, so these checks
  aren't necessary.

Found by: Coverity <http://scan.coverity.com>
(CID: 3)


r859586 | lundblad | 2006-05-04 14:45:54 -0500 (Thu, 04 May 2006)


* subversion/libsvn_ra_svn/editor.c
  (ra_svn_handle_apply_textdelta): Avoid use of uninitialized variable.

Found by: Coverity <http://www.coverity.com>
(CID: 28)


r859578 | djames | 2006-05-03 16:44:30 -0500 (Wed, 03 May 2006)

* subversion/libsvn_wc/props.c
  (svn_wc_props_modified_p): Cleanup dead code. We only need to check once
  whether an entry exists -- not three times! Also, cleanup subpools when no
  entry is found, and when propcaching is enabled.

Found by: Coverity <http://scan.coverity.com>
(CID: 2)


r859577 | djames | 2006-05-03 16:22:20 -0500 (Wed, 03 May 2006)

* subversion/libsvn_fs_base/dag.c
  (svn_fs_base__dag_commit_txn): Don't check whether 'new_rev' is NULL. We
  already require that new_rev must be non-NULL because we dereference the
  pointer regardless.

Found by: Coverity <http://scan.coverity.com>
(CID: 11)