diff options
author | John Szakmeister <john@szakmeister.net> | 2015-05-08 04:48:39 -0400 |
---|---|---|
committer | John Szakmeister <john@szakmeister.net> | 2015-05-09 09:28:38 -0400 |
commit | d3a57b9b0b16edb183fb57eeb0831ee42b204929 (patch) | |
tree | f7563f12bc8385f1546a567a045c904474002e08 /runtime/doc/manpages/ja/vim-ja.1 | |
parent | d00558bbdb828ddcd713446e6256d6c0d0430e04 (diff) | |
download | rneovim-d3a57b9b0b16edb183fb57eeb0831ee42b204929.tar.gz rneovim-d3a57b9b0b16edb183fb57eeb0831ee42b204929.tar.bz2 rneovim-d3a57b9b0b16edb183fb57eeb0831ee42b204929.zip |
Fix a wrong expectation for vim_FullName() and FullName_save().
It's the second argument, buf, that cannot be NULL. fname is allowed to
be NULL. The issue only showed up on the release build when trying to
use NULL for fname and the test would segfault unexpectedly (because the
NULL check for fname was being optimized out due to the function
attributes).
FullName_save() also incorrectly assumes that fname cannot be NULL
(possibly because of the attribute on vim_FullName), so fix that site as
well. This didn't have a corresponding test, so it wasn't visible as
test breakage, but did generate a tautological comparison warning in the
release build under Clang.
Diffstat (limited to 'runtime/doc/manpages/ja/vim-ja.1')
0 files changed, 0 insertions, 0 deletions