1,073
edits
PVS static analyzer - 5.0 Release: Difference between revisions
Jump to navigation
Jump to search
m
Mark bug fixed
m (Mark bug fixed) |
m (Mark bug fixed) |
||
| Line 10,618: | Line 10,618: | ||
</tr> | </tr> | ||
<tr> | <tr> | ||
<td style='width: 30%;'><div title="/home/pavel/octave/liboctave/util/lo-regexp.cc">lo-regexp.cc (127)</div></td> | <td style='width: 30%; text-decoration: line-through;'><div title="/home/pavel/octave/liboctave/util/lo-regexp.cc">lo-regexp.cc (127)</div></td> | ||
<td style='width: 5%;'>[https://www.viva64.com/en/w/v555/ V555]</td> | <td style='width: 5%;'>[https://www.viva64.com/en/w/v555/ V555]</td> | ||
<td style='width: 65%;'>The expression 'new_pos - pos > 0' will work as 'new_pos != pos'.</td> | <td style='width: 65%;'>The expression 'new_pos - pos > 0' will work as 'new_pos != pos'.</td> | ||
| Line 10,633: | Line 10,633: | ||
</tr> | </tr> | ||
<tr> | <tr> | ||
<td style='width: 30%;'><div title="/home/pavel/octave/liboctave/util/lo-utils.cc">lo-utils.cc (44)</div></td> | <td style='width: 30%; text-decoration: line-through;'><div title="/home/pavel/octave/liboctave/util/lo-utils.cc">lo-utils.cc (44)</div></td> | ||
<td style='width: 5%;'>[https://www.viva64.com/en/w/v550/ V550]</td> | <td style='width: 5%;'>[https://www.viva64.com/en/w/v550/ V550]</td> | ||
<td style='width: 65%;'>An odd precise comparison: octave::math::x_nint(x) == x. It's probably better to use a comparison with defined precision: fabs(A - B) < Epsilon.</td> | <td style='width: 65%;'>An odd precise comparison: octave::math::x_nint(x) == x. It's probably better to use a comparison with defined precision: fabs(A - B) < Epsilon.</td> | ||
</tr> | </tr> | ||
<tr> | <tr> | ||
<td style='width: 30%;'><div title="/home/pavel/octave/liboctave/util/lo-utils.cc">lo-utils.cc (47)</div></td> | <td style='width: 30%; text-decoration: line-through;'><div title="/home/pavel/octave/liboctave/util/lo-utils.cc">lo-utils.cc (47)</div></td> | ||
<td style='width: 5%;'>[https://www.viva64.com/en/w/v550/ V550]</td> | <td style='width: 5%;'>[https://www.viva64.com/en/w/v550/ V550]</td> | ||
<td style='width: 65%;'>An odd precise comparison: x == 0. It's probably better to use a comparison with defined precision: fabs(A - B) < Epsilon.</td> | <td style='width: 65%;'>An odd precise comparison: x == 0. It's probably better to use a comparison with defined precision: fabs(A - B) < Epsilon.</td> | ||
</tr> | </tr> | ||
<tr> | <tr> | ||
<td style='width: 30%;'><div title="/home/pavel/octave/liboctave/util/lo-utils.cc">lo-utils.cc (47)</div></td> | <td style='width: 30%; text-decoration: line-through;'><div title="/home/pavel/octave/liboctave/util/lo-utils.cc">lo-utils.cc (47)</div></td> | ||
<td style='width: 5%;'>[https://www.viva64.com/en/w/v550/ V550]</td> | <td style='width: 5%;'>[https://www.viva64.com/en/w/v550/ V550]</td> | ||
<td style='width: 65%;'>An odd precise comparison: x == 1. It's probably better to use a comparison with defined precision: fabs(A - B) < Epsilon.</td> | <td style='width: 65%;'>An odd precise comparison: x == 1. It's probably better to use a comparison with defined precision: fabs(A - B) < Epsilon.</td> | ||
</tr> | </tr> | ||
<tr> | <tr> | ||
<td style='width: 30%;'><div title="/home/pavel/octave/liboctave/util/lo-utils.cc">lo-utils.cc (50)</div></td> | <td style='width: 30%; text-decoration: line-through;'><div title="/home/pavel/octave/liboctave/util/lo-utils.cc">lo-utils.cc (50)</div></td> | ||
<td style='width: 5%;'>[https://www.viva64.com/en/w/v550/ V550]</td> | <td style='width: 5%;'>[https://www.viva64.com/en/w/v550/ V550]</td> | ||
<td style='width: 65%;'>An odd precise comparison: x == 0. It's probably better to use a comparison with defined precision: fabs(A - B) < Epsilon.</td> | <td style='width: 65%;'>An odd precise comparison: x == 0. It's probably better to use a comparison with defined precision: fabs(A - B) < Epsilon.</td> | ||
</tr> | </tr> | ||
<tr> | <tr> | ||
<td style='width: 30%;'><div title="/home/pavel/octave/liboctave/util/lo-utils.cc">lo-utils.cc (65)</div></td> | <td style='width: 30%; text-decoration: line-through;'><div title="/home/pavel/octave/liboctave/util/lo-utils.cc">lo-utils.cc (65)</div></td> | ||
<td style='width: 5%;'>[https://www.viva64.com/en/w/v550/ V550]</td> | <td style='width: 5%;'>[https://www.viva64.com/en/w/v550/ V550]</td> | ||
<td style='width: 65%;'>An odd precise comparison: octave::math::x_nint(x) == x. It's probably better to use a comparison with defined precision: fabs(A - B) < Epsilon.</td> | <td style='width: 65%;'>An odd precise comparison: octave::math::x_nint(x) == x. It's probably better to use a comparison with defined precision: fabs(A - B) < Epsilon.</td> | ||
</tr> | </tr> | ||
<tr> | <tr> | ||
<td style='width: 30%;'><div title="/home/pavel/octave/liboctave/util/lo-utils.cc">lo-utils.cc (68)</div></td> | <td style='width: 30%; text-decoration: line-through;'><div title="/home/pavel/octave/liboctave/util/lo-utils.cc">lo-utils.cc (68)</div></td> | ||
<td style='width: 5%;'>[https://www.viva64.com/en/w/v550/ V550]</td> | <td style='width: 5%;'>[https://www.viva64.com/en/w/v550/ V550]</td> | ||
<td style='width: 65%;'>An odd precise comparison: x == 0. It's probably better to use a comparison with defined precision: fabs(A - B) < Epsilon.</td> | <td style='width: 65%;'>An odd precise comparison: x == 0. It's probably better to use a comparison with defined precision: fabs(A - B) < Epsilon.</td> | ||
</tr> | </tr> | ||
<tr> | <tr> | ||
<td style='width: 30%;'><div title="/home/pavel/octave/liboctave/util/lo-utils.cc">lo-utils.cc (68)</div></td> | <td style='width: 30%; text-decoration: line-through;'><div title="/home/pavel/octave/liboctave/util/lo-utils.cc">lo-utils.cc (68)</div></td> | ||
<td style='width: 5%;'>[https://www.viva64.com/en/w/v550/ V550]</td> | <td style='width: 5%;'>[https://www.viva64.com/en/w/v550/ V550]</td> | ||
<td style='width: 65%;'>An odd precise comparison: x == 1. It's probably better to use a comparison with defined precision: fabs(A - B) < Epsilon.</td> | <td style='width: 65%;'>An odd precise comparison: x == 1. It's probably better to use a comparison with defined precision: fabs(A - B) < Epsilon.</td> | ||
</tr> | </tr> | ||
<tr> | <tr> | ||
<td style='width: 30%;'><div title="/home/pavel/octave/liboctave/util/lo-utils.cc">lo-utils.cc (71)</div></td> | <td style='width: 30%; text-decoration: line-through;'><div title="/home/pavel/octave/liboctave/util/lo-utils.cc">lo-utils.cc (71)</div></td> | ||
<td style='width: 5%;'>[https://www.viva64.com/en/w/v550/ V550]</td> | <td style='width: 5%;'>[https://www.viva64.com/en/w/v550/ V550]</td> | ||
<td style='width: 65%;'>An odd precise comparison: x == 0. It's probably better to use a comparison with defined precision: fabs(A - B) < Epsilon.</td> | <td style='width: 65%;'>An odd precise comparison: x == 0. It's probably better to use a comparison with defined precision: fabs(A - B) < Epsilon.</td> | ||
</tr> | </tr> | ||
<tr> | <tr> | ||
<td style='width: 30%;'><div title="/home/pavel/octave/liboctave/util/lo-utils.cc">lo-utils.cc (113)</div></td> | <td style='width: 30%; text-decoration: line-through;'><div title="/home/pavel/octave/liboctave/util/lo-utils.cc">lo-utils.cc (113)</div></td> | ||
<td style='width: 5%;'>[https://www.viva64.com/en/w/v575/ V575]</td> | <td style='width: 5%;'>[https://www.viva64.com/en/w/v575/ V575]</td> | ||
<td style='width: 65%;'>The potential null pointer is passed into 'fgets' function. Inspect the first argument. Check lines: 113, 104.</td> | <td style='width: 65%;'>The potential null pointer is passed into 'fgets' function. Inspect the first argument. Check lines: 113, 104.</td> | ||
</tr> | </tr> | ||
<tr> | <tr> | ||
<td style='width: 30%;'><div title="/home/pavel/octave/liboctave/util/lo-utils.cc">lo-utils.cc (129)</div></td> | <td style='width: 30%; text-decoration: line-through;'><div title="/home/pavel/octave/liboctave/util/lo-utils.cc">lo-utils.cc (129)</div></td> | ||
<td style='width: 5%;'>[https://www.viva64.com/en/w/v769/ V769]</td> | <td style='width: 5%;'>[https://www.viva64.com/en/w/v769/ V769]</td> | ||
<td style='width: 65%;'>The 'buf' pointer in the 'buf + tmp' expression could be nullptr. In such case, resulting value will be senseless and it should not be used. Check lines: 129, 122.</td> | <td style='width: 65%;'>The 'buf' pointer in the 'buf + tmp' expression could be nullptr. In such case, resulting value will be senseless and it should not be used. Check lines: 129, 122.</td> | ||
| Line 10,753: | Line 10,753: | ||
</tr> | </tr> | ||
<tr> | <tr> | ||
<td style='width: 30%;'><div title="/home/pavel/octave/liboctave/util/oct-inttypes.cc">oct-inttypes.cc (784)</div></td> | <td style='width: 30%; text-decoration: line-through;'><div title="/home/pavel/octave/liboctave/util/oct-inttypes.cc">oct-inttypes.cc (784)</div></td> | ||
<td style='width: 5%;'>[https://www.viva64.com/en/w/v524/ V524]</td> | <td style='width: 5%;'>[https://www.viva64.com/en/w/v524/ V524]</td> | ||
<td style='width: 65%;'>It is odd that the body of 'powf' function is fully equivalent to the body of 'pow' function.</td> | <td style='width: 65%;'>It is odd that the body of 'powf' function is fully equivalent to the body of 'pow' function.</td> | ||
| Line 10,833: | Line 10,833: | ||
</tr> | </tr> | ||
<tr> | <tr> | ||
<td style='width: 30%;'><div title="/home/pavel/octave/libgui/qterminal/libqterminal/unix/CharacterColor.h">CharacterColor.h (44)</div></td> | <td style='width: 30%; text-decoration: line-through;'><div title="/home/pavel/octave/libgui/qterminal/libqterminal/unix/CharacterColor.h">CharacterColor.h (44)</div></td> | ||
<td style='width: 5%;'>[https://www.viva64.com/en/w/v690/ V690]</td> | <td style='width: 5%;'>[https://www.viva64.com/en/w/v690/ V690]</td> | ||
<td style='width: 65%;'>The 'ColorEntry' class implements the '=' operator, but lacks a copy constructor. It is dangerous to use such a class.</td> | <td style='width: 65%;'>The 'ColorEntry' class implements the '=' operator, but lacks a copy constructor. It is dangerous to use such a class.</td> | ||