Menu

#309 bug in ServiceClassUser::SendFind in master branch

2.4.1
closed
None
5
2013-11-28
2013-11-28
No

Compiler warning: no break statement after case 0xA900.

So, this case will fall through to 0xA700.

See below:

Thanks,
Aaron

///////////////////////////////////////////////////////////////////////////////////////////////////////
case 0xA900: // Identifier Does Not Match SOP Class
{
Attribute<0x0,0x0901> errormsg;
if( ds.FindDataElement( errormsg.GetTag() ) )
{
errormsg.SetFromDataSet( ds );
gdcm::Tag const & t = errormsg.GetValue();
gdcmErrorMacro( "Offending Element: " << t ); (void)t;
}
else
{
gdcmErrorMacro( "Offending Element ??" );
}
}
case 0xA700: // Refused: Out of Resources

Discussion

  • Mathieu Malaterre

    [release-2-2 be1cf30] Missing break statement in switch
    Author: Aaron Boxer boxerab@gmail.com
    1 file changed, 1 insertion(+)

     
  • Mathieu Malaterre

    • status: open --> closed
    • Group: 2.3 --> 2.4.1
     

Log in to post a comment.