[ovs-dev] [PATCH v4] Makefile.am: Add clang static analysis support

Bodireddy, Bhanuprakash bhanuprakash.bodireddy at intel.com
Fri Jul 15 18:48:36 UTC 2016


>> +Open vSwitch includes a Makefile target to trigger static code
>> +Analysis and the instructions are below.
>> +
>> +1. ./boot.sh
>> +2. ./configure CC=clang (when using clang compiler)
>> +   ./configure CC=gcc CFLAGS="-std=gnu99" (when using GCC) 3. make
>> +clang-analyze
>
>OK, the above works for me now.  Thanks.
>
>> +4. scan-view --host=IPADDR host --port PORT
>> +    $OVS_DIR>/clang-analyzer-results/yyyy-mm-dd-114251-1027-1
>> +--allow-all-hosts
>
>The above doesn't seem right.  "host" is spurious and the specific digits you
>include are different (I think that they are the time of day).
>
>The last line of output from "make clang-analyze" lists a command to
>run:
>
>    scan-build: Run 'scan-view /home/blp/nicira/ovs/tests/clang-analyzer-
>results/2016-07-14-091820-11158-1' to examine bug reports.
>
>It's probably easiest to advise the user to just run that command.
>
>> +5. Visit http://ipaddr:PORT/ for analysis report.
>
>For me, at least, when I run the command above, the URL automatically pops
>up in my web browser, so that I don't have to do anything extra.

Hello Ben,
I have sent out V5 patch with the modifications you suggested in this thread.

Regards,
Bhanu Prakash.



More information about the dev mailing list