On Sat, Oct 05, 2013 at 12:16:07PM +0200, Alexander Chemeris wrote:
#define SUBSCR_PUT(sub) \
- sub->net = &dummy_net; \
- subscr_put(sub);
+ if (sub) { \
+ sub->net = &dummy_net; \
+ subscr_put(sub); \
+ }
I don't like this semantic change. Either the tests expects a valid
subscriber or it doesn't. It is not a "maybe" this query will result
in a look up.