[PATCH] D55823: [analyzer] Fix backward compatibility issue after D53280 'Emit an error for invalid -analyzer-config inputs'

Artem Dergachev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Dec 20 15:46:07 PST 2018


NoQ added a comment.

In D55823#1338337 <https://reviews.llvm.org/D55823#1338337>, @arphaman wrote:

> I think `Args.getArgString` might return `nullptr` so that's why you could crash.


Nope, we crash on index lookup. It seems that `Args.size()` is larger than the small vector that we're allowed to access via `.getArgStrings()`. I tried `Args.getNumInputArgStrings()` and it seems better.


Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D55823/new/

https://reviews.llvm.org/D55823





More information about the cfe-commits mailing list