Attention is currently required from: Thomas Heijligen, Anastasia Klimchuk, Nikolai Artemiev, Joursoir.

Patch set 1:Code-Review +1

View Change

1 comment:

• File cli_classic.c:

  • Patch Set #1, Line 1103: if (set_wp_region && wp_region)

    The standard says: `If ptr is a null pointer, no action occurs.` […]

    Given that `wp_region` is explicitly first-time initialised to NULL (in the same line as the declaration), I'd free it the same way the other pointers are freed:

     free(wp_region);

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