[gmx-developers] jenkins cppcheck picky?
David van der Spoel
spoel at xray.bmc.uu.se
Sun Sep 1 12:44:42 CEST 2013
On 2013-09-01 10:17, David van der Spoel wrote:
> On 2013-09-01 01:23, Mark Abraham wrote:
>> We could... Jenkins master was using 1.53. I compiled latest cppcheck
>> 1.61 and installed it in /opt/local, but see
>> from using it. Using 1.61 routinely will need someone to clean those
>> up. Don't look at me for at least a fortnight :-)
> I can give it a try.
> At least those look more like serious problems.
On second thoughts, some of those new ones look fishy too...
Mismatching assignment and comparison, comparison 'c==4' is always
false. (line 301):
0294 for (i = 0; i < na; i++)
0296 xnb[j+XX*PACK_X4] = x[a[i]][XX];
0297 xnb[j+YY*PACK_X4] = x[a[i]][YY];
0298 xnb[j+ZZ*PACK_X4] = x[a[i]][ZZ];
0301 if (c == PACK_X4)
0303 j += (DIM-1)*PACK_X4;
0304 c = 0;
> We can not use formatted input because it works the same way in C as in
> fortran, reading a fixed number of columns.
>> On Sat, Aug 31, 2013 at 9:33 PM, David van der Spoel
>> <spoel at xray.bmc.uu.se> wrote:
>>> I'm getting complaints in jenkins from cppcheck, for code like
>>> int d;
>>> fscanf(fp, "%d", &d);
>>> scanf without field width limits can crash with huge input data
>>> Apparently this is a check for an old glibc or something:
>>> Can we upgrade cppcheck?
>>> David van der Spoel, Ph.D., Professor of Biology
>>> Dept. of Cell & Molec. Biol., Uppsala University.
>>> Box 596, 75124 Uppsala, Sweden. Phone: +46184714205.
>>> spoel at xray.bmc.uu.se http://folding.bmc.uu.se
>>> gmx-developers mailing list
>>> gmx-developers at gromacs.org
>>> Please don't post (un)subscribe requests to the list. Use the www
>>> or send it to gmx-developers-request at gromacs.org.
David van der Spoel, Ph.D., Professor of Biology
Dept. of Cell & Molec. Biol., Uppsala University.
Box 596, 75124 Uppsala, Sweden. Phone: +46184714205.
spoel at xray.bmc.uu.se http://folding.bmc.uu.se
More information about the gromacs.org_gmx-developers