23
66%
master: 54%

Ran 17 Jul 2017 12:54PM UTC

Jobs 1

Files 65

Run time 3s

Badge

Embed ▾

pending completion

Build # 23

Build Type

push

travis-ci

Committed by

coldfix

Commit Message

Fix growth of zero-length arrays

In many places the `grow_XXX_array` functions are used to allocate an
additional array element directly before writing into the supposedly
allocated array index without further range-checks.

Now, if an array was initialized to zero length it was never grown,
meaning that we would write after the end of the allocated buffer.

This just caught me with the `select->at` parameter which has to have
zero length by default (as defined in mad_dict.c). If you specify one or
two values for `at`, everything *seemed* to work as expected. However,
`decode_par` did not properly grow the double array and was writing the
values after the end of the buffer. When specifying a third argument,
the problem manifested by writing a NUL into the first byte of the
`flag` buffer and hence not anymore executing any selection routine.

I need at least grow_double_array/grow_expr_list to make this work.

Note: we could alternatively fix this by always allocating at least an
`max=1` array to begin with.

Run Details

11901 of 17938 relevant lines covered (66.35%)

807414.33 hits per line

Jobs

ID	Job ID	Ran	Files	Coverage
1	23.1	17 Jul 2017 12:54PM UTC	0	66.35	Travis Job 23.1

Source Files on build 23

Detailed source file information is not available for this build.

hibtc / MAD-X / 23
66%
master: 54%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 23

hibtc / MAD-X / 23 66% master: 54%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 23

hibtc / MAD-X / 23
66%
master: 54%

README BADGES
x