Attention is currently required from: laforge.

View Change

1 comment:

• File src/common/osmux.c:

  • Patch Set #1, Line 141: TALLOC_FREE

    why uppercase? Don't recall having seen this before... […]

    it's a standard talloc API (define) and already used in several places. It calls talloc_free and sets the pointer to NULL.
    I know the h structure is being freed immediately afterwards, but I prefer setting this as NULL in order to avoid weird accesses in the future in case there's some talloc destructor set in the parent "h" structure.

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