From fbafb8f249b5e485ff7ec539eb614aa42a9a647e Mon Sep 17 00:00:00 2001 From: Josh Oguin Date: Wed, 19 Nov 2014 14:46:43 -0600 Subject: chainimpl.h: Add _Assert() to _Chain_Initialize_empty() CodeSonar flagged this as a potential NULL deference. That should never occur but adding the _Assert() ensures we are checking that. --- cpukit/score/include/rtems/score/chainimpl.h | 12 +++++++++--- 1 file changed, 9 insertions(+), 3 deletions(-) diff --git a/cpukit/score/include/rtems/score/chainimpl.h b/cpukit/score/include/rtems/score/chainimpl.h index 6e1b6e3ce5..08cbab6bce 100644 --- a/cpukit/score/include/rtems/score/chainimpl.h +++ b/cpukit/score/include/rtems/score/chainimpl.h @@ -7,7 +7,7 @@ /* * Copyright (c) 2010 embedded brains GmbH. * - * COPYRIGHT (c) 1989-2006. + * COPYRIGHT (c) 1989-2014. * On-Line Applications Research Corporation (OAR). * * The license and distribution terms for this file may be @@ -20,6 +20,7 @@ #include #include +#include #ifdef __cplusplus extern "C" { @@ -613,8 +614,13 @@ RTEMS_INLINE_ROUTINE void _Chain_Initialize_empty( Chain_Control *the_chain ) { - Chain_Node *head = _Chain_Head( the_chain ); - Chain_Node *tail = _Chain_Tail( the_chain ); + Chain_Node *head; + Chain_Node *tail; + + _Assert( the_chain != NULL ); + + head = _Chain_Head( the_chain ); + tail = _Chain_Tail( the_chain ); head->next = tail; head->previous = NULL; -- cgit v1.2.3