Project

General

Profile

Actions

Bug #236

closed

Bug #235: meta: Missing list checks for *list_add*

batadv_gw_node_update: Missing list checks for *list_add*

Added by Sven Eckelmann about 8 years ago. Updated almost 4 years ago.

Status:
Closed
Priority:
Normal
Target version:
Start date:
06/26/2015
Due date:
% Done:

0%

Estimated time:

Description

Simon debugged the refcnt problem and submitted some patches to fix them. I had a brief look and noticed that there are possible more problems similar to the *list_del* ones - just with *list_add*. Basically some functions use some kind of get function, notice that the element does not exist and then create a new one to add to the list. Only the "list_add" is protected. The result may be that an element in twice in a list when only a single occurrence is allowed.

The problem I saw is that functions adding objects in an RCU protected list are missing an definitive check. They first call some kind of *_get (rcu_read_lock only) to check if an object with this value already exists and then uses some kind of *_add to allocate a new object and add it (which may already be added in by a different context). So it has to be made sure that nothing modifies the list between the check and the add of the new object).

An RFC was submitted to demonstrate how it could be fixed https://patchwork.open-mesh.org/project/b.a.t.m.a.n./patch/1898798.WP4bC1arXA@bentobox/

Actions #1

Updated by Sven Eckelmann about 7 years ago

  • Assignee changed from Marek Lindner to batman-adv developers
Actions #3

Updated by Sven Eckelmann over 5 years ago

  • Status changed from In Progress to Resolved
  • Target version set to 2018.3

Queued up for 2018.3

Actions #4

Updated by Sven Eckelmann over 5 years ago

  • Status changed from Resolved to Closed
Actions #5

Updated by Sven Eckelmann almost 4 years ago

  • Description updated (diff)
Actions

Also available in: Atom PDF