Re: [devel] Помогите побороть ошибку в gdcm

Re: [devel] Помогите побороть ошибку в gdcm

[Ivan A. Melnikov]

[-- Attachment #1: Type: text/plain, Size: 977 bytes --]

On Sun, 27 Nov 2011 03:24:50 -0500
Dubrovskiy Viacheslav <slava@tangramltd.com> wrote:

> Здравствуйте
> 
> Пытаюсь собрать http://sourceforge.net/projects/gdcm/
> DiCoM is a C++ library for DICOM medical files
> 
> При сборке получаю вот такую ошибку. Смотрел спеки в федоре, дебиане
> - у них все собирается без дополнительных патчей.
> Помогите исправить плиз.
[...]

Вариант патча в аттаче. Переполнение буфера там правда есть, gcc прав.
Советую связаться с апстримом, тем более что, хотя моё исправление
выглядит надёжным для этой проблемы, что конкретно *должен* делать этот
код я не разбирался.

-- 
WBR,
Ivan A. Melnikov

[-- Attachment #2: gdcm-2.0.18-alt-fix-overflow.patch --]
[-- Type: text/x-patch, Size: 555 bytes --]

diff --git a/Source/DataStructureAndEncodingDefinition/gdcmElement.h b/Source/DataStructureAndEncodingDefinition/gdcmElement.h
index 54e3796..9e8f176 100644
--- a/Source/DataStructureAndEncodingDefinition/gdcmElement.h
+++ b/Source/DataStructureAndEncodingDefinition/gdcmElement.h
@@ -690,7 +690,7 @@ template<>
 class Element<VR::AS, VM::VM5>
 {
 public:
-  char Internal[VMToLength<VM::VM5>::Length];
+  char Internal[VMToLength<VM::VM5>::Length * sizeof( VRToType<VR::AS>::Type )];
   void Print(std::ostream &_os) const {
     _os << Internal;
     }