Project

General

Profile

Bug #242

batadv_neigh_node_release: Double batadv_hardif_neigh_put

Added by Sven Eckelmann over 5 years ago. Updated over 4 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Target version:
Start date:
03/05/2016
Due date:
% Done:

0%

Estimated time:

Description

batadv_neigh_node_release (and maybe other places) uses a get + a double put of batadv_hardif_neigh_put without a direct reference in batadv_hardif_neigh_node. So it could be that the reference which is free'd is actually for a different object in memory or maybe was never obtained.

It looks like there should be an explicit reference (pointer) in batadv_hardif_neigh_node to be sure that the correct reference is free'd.


Related issues

Related to batman-adv - Bug #237: batadv_neigh_node_new: Missing list checks for *list_add*Closed06/26/2015

Actions

History

#1

Updated by Sven Eckelmann over 5 years ago

  • Related to Bug #237: batadv_neigh_node_new: Missing list checks for *list_add* added
#2

Updated by Sven Eckelmann over 5 years ago

Proof-of-concept patches are available in ecsv/no_double_trouble

RFC of the patch was submitted to the mailing list as https://patchwork.open-mesh.org/project/b.a.t.m.a.n./patch/1457258842-10389-2-git-send-email-sven@narfation.org/

#3

Updated by Sven Eckelmann over 5 years ago

  • Status changed from New to In Progress

I've posted the cleaned up version of the RFC as patch because there was no objection regarding the idea behind the fix.

#4

Updated by Marek Lindner over 5 years ago

  • Status changed from In Progress to Closed

Sven Eckelmann wrote:

I've posted the cleaned up version of the RFC as patch because there was no objection regarding the idea behind the fix.

Merged! Thanks!

#5

Updated by Sven Eckelmann over 4 years ago

  • Target version set to 2016.1

Also available in: Atom PDF