bug in poll/connect

All other questions regarding DCMTK

Moderator: Moderator Team

Post Reply
Message
Author
Shaeto
Posts: 147
Joined: Tue, 2009-01-20, 17:50
Location: CA, USA
Contact:

bug in poll/connect

#1 Post by Shaeto »

Hi, there is an important request on github (not my)

https://github.com/DCMTK/dcmtk/pull/3

i am not sure about changes for dcmtrans.cc but POLLIN in dcmfsm.cc is very serious issue, 3.6.3 release is not usable under linux because of poll(..POLLIN...) after connect(), it should be POLLOUT

Thanks!

Marco Eichelberg
OFFIS DICOM Team
OFFIS DICOM Team
Posts: 1444
Joined: Tue, 2004-11-02, 17:22
Location: Oldenburg, Germany
Contact:

Re: bug in poll/connect

#2 Post by Marco Eichelberg »

Thanks for your posting. We are aware of the pull request, and it's on the "to do list" (I think Michael Onken is working on it currently).

Michael Onken
DCMTK Developer
Posts: 2049
Joined: Fri, 2004-11-05, 13:47
Location: Oldenburg, Germany
Contact:

Re: bug in poll/connect

#3 Post by Michael Onken »

The POLLOUT versus POLLIN patch is already in DCMTK, I just pushed it to the public master branch, see this commit. Thanks for pointing us to the problem Shaeto.

The other part of the related github pull request is not yet reviewed/patched.

Best,
Michael

Post Reply

Who is online

Users browsing this forum: Bing [Bot] and 1 guest