Attention is currently required from: Felix Singer, Nico Huber, Angel Pons, Light, Anastasia Klimchuk.

Patch set 10:Code-Review +2

View Change

1 comment:

• File ich_descriptors.c:

  • Patch Set #7, Line 1364: int ret = read_ich_descriptors_from_dump(dump, len, &cs, &desc);

    It could be a false-positive (in a project where tools like scan-build find little […]

    Not sure if it actually covers up a bug Nico, more that it would fail predictably in the same manner if there was a bug rather than - happens to have zero in memory this time.

    I get what your thinking here but types having default values at instantiation I wouldn't say shadows bugs.

    However, one could argue read_ich_descriptors_from_dump() should memset to zero before filling in fields so that the domain of the function is always well defined in desc. Depending on how one wishes to define their conventions of responsibilities.

To view, visit change 62763. To unsubscribe, or for help writing mail filters, visit settings.