20 Jun
2016
20 Jun
'16
8:14 p.m.
On Monday 20 June 2016 21:39:54 Linus Lüssing wrote:
This patch abstracts the forward packet creation into the new function batadv_forw_packet_alloc().
The queue counting and interface reference counters are now handled internally within batadv_forw_packet_alloc() and its batadv_forw_packet_free() counterpart. This should reduce the risk of having reference/queue counting bugs again and should increase code readibility.
Signed-off-by: Linus Lüssing linus.luessing@c0d3.blue
Changelog v4:
- Removed goto in batadv_iv_ogm_aggregate_new()
- Removed two goto's in batadv_forw_packet_alloc()
- Easier to read debug output code in batadv_forw_packet_alloc()
- Renamed "out" and "packet_free" jump labels in batadv_add_bcast_packet_to_list() to "err" and "err_packet_free"
Reviewed-by: Sven Eckelmann sven@narfation.org
Kind regards, Sven