RTOS/rtems
1
0
Fork 0
forked from Imagelibrary/rtems
Code Pull Requests Activity

chainimpl.h: Add _Assert() to _Chain_Initialize_empty()

Browse Source 
CodeSonar flagged this as a potential NULL deference. That should never
occur but adding the _Assert() ensures we are checking that.
This commit is contained in:
Josh Oguin
2014-11-19 14:46:43 -06:00
committed by Joel Sherrill
parent e106aa7380
commit fbafb8f249
1 changed files with 9 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
cpukit/score/include/rtems/score/chainimpl.h
View File

@@ -7,7 +7,7 @@
/* /*
* Copyright (c) 2010 embedded brains GmbH. * Copyright (c) 2010 embedded brains GmbH.
* *
* COPYRIGHT (c) 1989-2006. * COPYRIGHT (c) 1989-2014.
* On-Line Applications Research Corporation (OAR). * On-Line Applications Research Corporation (OAR).
* *
* The license and distribution terms for this file may be * The license and distribution terms for this file may be
@@ -20,6 +20,7 @@
#include <rtems/score/chain.h> #include <rtems/score/chain.h>
#include <rtems/score/address.h> #include <rtems/score/address.h>
#include <rtems/score/assert.h>
#ifdef __cplusplus #ifdef __cplusplus
extern "C" { extern "C" {
@@ -613,8 +614,13 @@ RTEMS_INLINE_ROUTINE void _Chain_Initialize_empty(
Chain_Control *the_chain Chain_Control *the_chain
) )
{ {
Chain_Node *head = _Chain_Head( the_chain ); Chain_Node *head;
Chain_Node *tail = _Chain_Tail( the_chain ); Chain_Node *tail;
_Assert( the_chain != NULL );
head = _Chain_Head( the_chain );
tail = _Chain_Tail( the_chain );
head->next = tail; head->next = tail;
head->previous = NULL; head->previous = NULL;