Project

General

Profile

Actions

Bug #3701

closed

rate_ctr_group_alloc() may return NULL if a rate counter group name is not up to specifications; make sure all callers handle that.

Added by neels over 5 years ago. Updated almost 5 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
Target version:
-
Start date:
11/20/2018
Due date:
% Done:

100%

Spec Reference:

Description

At least libosmo-mgcp/mgcp_protocol.c has various invocations of rate_ctr_group_alloc() that ignore the possibility that the name might be wrong and the function would return NULL then.
There should be checks for each invocation; if we some day choose to change the names and/or the rules for names, the code should warn about it instead of segfaulting randomly.
grep for rate_ctr_group_alloc() in the code base and make sure each caller can handle a NULL return value in a sane way.

Actions #2

Updated by laforge almost 5 years ago

  • Assignee set to laforge
Actions #3

Updated by laforge almost 5 years ago

  • Status changed from New to In Progress
  • % Done changed from 0 to 80

searched through all our repositories and submitted some related fixes to gerrit:

Actions #4

Updated by laforge almost 5 years ago

  • Status changed from In Progress to Resolved
  • % Done changed from 80 to 100

patches merged

Actions

Also available in: Atom PDF

Add picture from clipboard (Maximum size: 48.8 MB)