Re: [itom-discussions] Buffer overflow in SerialIO

[Marc G. <mar...@it...>]

Hallo Herr Kipp,

auf den ersten Blick teile ich Ihre Einschätzung, dass das Vorzeichen im
Linux-Teil von sread bei
*len = *len > readable ? *len : readable;
falsch ist. Ich kann das gerne korrigieren. Wenn Sie das gerade mit einem
Linux-System testen könnten, wäre das perfekt.

Was meinen Sie mit er Prüfung der Länge des Buffers im Falle von sendDelay?
Haben Sie Änderungsvorschlag im Quelltext,
um das mögliche Problem zu beheben? Sie können hierzu auch gerne ein Issue
im plugins-Repository anlegen und evtl. sogar
ein Pull Request zur Verfügung stellen.

Mit freundlichen Grüßen
Marc Gronle

Am Mo., 24. Sep. 2018 um 15:00 Uhr schrieb Thomas Kipp <ki...@tw...>:

> Hallo,
>
>
>
> in *SerialIO.cpp* Zeile 757ff (sRead) sieht es für mich aus, als würde
> die Größe des Buffers in den meisten Fällen nicht beachtet.
>
> Unter #ifndef WIN32 wird zunächst die Länge durch “ *len = *len >
> readable ? *len : readable;“ begrenzt, wobei allerdings immer der größere
> Wert übernommen wird unter #else ist das andersrum. In beiden Fällen wird
> im Fall von „sendDelay“ die Länge des Buffers überhaupt nicht beachtet.
>
>
>
>
>
> Mfg
>
> Thomas
>
>
> <https://www.avast.com/sig-email?utm_medium=email&utm_source=link&utm_campaign=sig-email&utm_content=emailclient> Virenfrei.
> www.avast.com
> <https://www.avast.com/sig-email?utm_medium=email&utm_source=link&utm_campaign=sig-email&utm_content=emailclient>
> <#m_6382713539044308119_DAB4FAD8-2DD7-40BB-A1B8-4E2AA1F9FDF2>
> _______________________________________________
> Itom-discussions mailing list
> Ito...@li...
> https://lists.sourceforge.net/lists/listinfo/itom-discussions
>


-- 
======================================================================
Marc Gronle <gr...@it...>
Institut für Technische Optik, Universität Stuttgart
======================================================================