Skip to content

Commit f55f3c8

Browse files
michaelonkenjcfr
authored andcommitted
[PATCH] Fixed segment access via Segment Number.
Fixed segment access via Segment Number which now assumes that the first Segment Number is not 0 as in the underlying vector, but starts with 1. Thanks to Andrey Fedorov <andrey.fedorov@gmail.com> for the report.
1 parent 545de87 commit f55f3c8

File tree

1 file changed

+4
-2
lines changed

1 file changed

+4
-2
lines changed

dcmseg/libsrc/segdoc.cc

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -664,7 +664,8 @@ DcmSegment* DcmSegmentation::getSegment(const size_t& segmentNumber)
664664
{
665665
return NULL;
666666
}
667-
return m_Segments[segmentNumber];
667+
// logical segment numbering starts with 1, so subtract 1 for vector index
668+
return m_Segments[segmentNumber-1];
668669
}
669670

670671

@@ -676,7 +677,8 @@ OFBool DcmSegmentation::getSegmentNumber(const DcmSegment* segment,
676677
{
677678
if (m_Segments.at(count) == segment)
678679
{
679-
segmentNumber = count;
680+
// logical segment numbering starts with 1 but vector index with 0
681+
segmentNumber = count + 1;
680682
return OFTrue;
681683
}
682684
}

0 commit comments

Comments
 (0)