Bug #594
closed
getMACAddress() crashes on Windows 8 / 10
100%
Description
See: http://forum.dcmtk.org/viewtopic.php?f=1&t=3971&p=15880
This function should be completely rewritten, at least the Windows-specific part, and tested on all Windows versions starting from Windows 2000, i.e. we should drop support for Windows 95 (and probably Windows 98/Me), which makes life - hopefully - a little easier.
Files
Updated by OFFIS DICOM Team about 11 years ago
lhfcjhyy(at)gmail.com writes:
Recently I got a bug when I use dcmGenerateUniqueIdentifier() under dcuid.cc. It just crashed when I use it, but it doesn't happen all the time. It works on my computer, but doesn't on the computer of a tester. Both of us OS is 64 bit windows 8 and running 32 bit program. The version of dcmtk I am using is dcmtk-3.6.1_20121102.
So when I trying to trace the bug , and I find the program crashed at line 1389 of dcuid.cc:ret = m_Query(SNMP_PDU_GETNEXT, &varBindList, &errorStatus, &errorIndex);
Updated by Marco Eichelberg almost 11 years ago
Update by 李海峰 on 2014-08-07:
we just fix the bug by update the OS from windows 8 to windows 8.1. So it's Microsoft's fault. Just want you know that, and maybe you can inform anyone else if they have this bug too :)
Updated by Jörg Riesmeier over 9 years ago
Also see this forum thread: http://forum.dcmtk.org/viewtopic.php?f=1&t=4254
Updated by Jörg Riesmeier over 9 years ago
- Status changed from New to Resolved
- Operating System set to Windows 8 (and newer)
Should be fixed with commit 64bbef6.
Updated by Jörg Riesmeier over 9 years ago
- Status changed from Resolved to Closed
- % Done changed from 0 to 100
Updated by Michael Onken over 8 years ago
- Status changed from Closed to Reopened
Verified today on Windows 10, Visual Studio 2015, 32 Bit Code Generation, static (non-DLL) build, also after the commit (64bbef6) a year ago.
The Codeguru page shows alternative approaches (especially method 3)
Updated by Michael Onken over 8 years ago
- Subject changed from getMACAddress() crashes on Windows 8 to getMACAddress() crashes on Windows 8 / 10
Updated by Marco Eichelberg over 8 years ago
- Target version changed from 3.6.1+ to 3.6.2
Updated by Marco Eichelberg over 8 years ago
- Status changed from Reopened to Closed
- Assignee changed from Michael Onken to Marco Eichelberg
Closed by commit 7c8f025.