aboutsummaryrefslogtreecommitdiff
path: root/src/nvim/diff.c
diff options
context:
space:
mode:
authorEliseo Martínez <eliseomarmol@gmail.com>2014-11-08 13:04:35 +0100
committerEliseo Martínez <eliseomarmol@gmail.com>2014-11-11 01:43:12 +0100
commitea1f883b199e957134bcb115184280e7122cadbb (patch)
treebfbfd72c4cb0fe703ee63d5b4d9c2404ca1a40ab /src/nvim/diff.c
parent1575642b11a48005563b9fc993325ed6e50d5488 (diff)
downloadrneovim-ea1f883b199e957134bcb115184280e7122cadbb.tar.gz
rneovim-ea1f883b199e957134bcb115184280e7122cadbb.tar.bz2
rneovim-ea1f883b199e957134bcb115184280e7122cadbb.zip
Fix warnings: ex_eval.c: report_pending(): Np dereference: FP.
Problem : Dereference of null pointer @ 711. Diagnostic : False positive. Rationale : Codepath producing error invokes this function with values `action=RPC_DISCARD, pending=CSTP_FINISH, value=NULL`. Now, for some reason, the analyzer is remembering that `value` is null, and that `action` is `RPC_DISCARD`, but it's not remembering that `pending` is `CSTP_FINISH`. Then, it's taking the wrong branch in the switch for `pending`. That path would never occur invocating the function with those values. Resolution : Assert function precondition between `pending` and `value`. This is, let the compiler know that `value` being null implies `pending` not containing `CSTP_THROW`.
Diffstat (limited to 'src/nvim/diff.c')
0 files changed, 0 insertions, 0 deletions