diff options
| author | Christophe Fergeau <cfergeau@redhat.com> | 2014-01-20 12:06:35 +0100 |
|---|---|---|
| committer | Christophe Fergeau <cfergeau@redhat.com> | 2014-01-20 14:33:33 +0100 |
| commit | 96ca358669cd32d17ce51f30de3cdbf0a1c0518c (patch) | |
| tree | 107357826aa73bd683e3b571dc6847585b02a286 /python_modules | |
| parent | 6175014ed77be6d1acd6420e36ca6582b573532c (diff) | |
| download | spice-common-96ca358669cd32d17ce51f30de3cdbf0a1c0518c.tar.gz spice-common-96ca358669cd32d17ce51f30de3cdbf0a1c0518c.tar.xz spice-common-96ca358669cd32d17ce51f30de3cdbf0a1c0518c.zip | |
quic: Fix test which is always true
find_model_params() is first doing *nbuckets = 0; and it then checks
nbuckets for NULL. This is redundant as the dereferencing would cause a
segfault if nbuckets was NULL, so the if (nbuckets) test can't be false.
As Uri pointed out, the "/* bucket start */" comment on the same line
probably implies that the test was meant to be 'if (*nbuckets)'
I've ran a few test and I did not observe issues because of it...
Diffstat (limited to 'python_modules')
0 files changed, 0 insertions, 0 deletions