Attention is currently required from: Anastasia Klimchuk, Peter Marheine.

View Change

1 comment:

• File sfdp.c:

  • Patch Set #6, Line 457: return 0;

    Had to also add free(tbuf) so that we don't create a memory leak.

    I just realized that this doesn't catch the failure if we have no Jedec parameter page at at all. I went back to the way it was handeled originally.
    Since this page being first is mandatory by the standard we can just fail in case we can't find it at position 0.

    This also solves some other errors in the way the page is parsed, because it sets feature bits instead of or-ing them. This would invalidate pages parsed beforehand, if the set feature bits

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