<div dir="ltr"><div style>First, I'll quote a paragraph from</div><div style><a href="https://github.com/erlang/otp/wiki/Submitting-patches">https://github.com/erlang/otp/wiki/Submitting-patches</a>:</div><div style><br>
</div><div style>"If you submit a patch in a way other than what is described here,</div><div style>it implies more work for the Erlang/OTP team. In that case we</div><div style>cannot guarantee anything. It is up to the maintainer for that</div>
<div style>part of OTP to decide whether (s)he will pick up the patch."</div><div style><br></div><div style>The patch also does not include a test case for the new</div><div style>functionality and has some code issues (e.g. "%" comments</div>
<div style>when the convention is to use "%%" comments). Therefore</div><div style>additional work is needed before it is ready for inclusion.</div><div style><br></div><div style>Moving on to the problem that the patch addresses,</div>
<div style>the use case is not clear to us.</div><div style><br></div><div style>If a user has access to an Erlang shell,<br></div><div style>there are many other ways in which (s)he</div><div style>could bring down the system. Therefore, it is not</div>
<div style>clear that you would gain very much by fixing this</div><div style>particular issue.</div><div style><br></div><div style>For these reasons, we reject the patch.</div><div style><br></div><div style>/Bjorn</div>
<div style><br></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, May 7, 2013 at 4:22 PM, Stefan Zegenhagen <span dir="ltr"><<a href="mailto:stefan.zegenhagen@arcutronix.com" target="_blank">stefan.zegenhagen@arcutronix.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Dear all,<br>
<br>
<br>
it seems as if I sent around the wrong version of the patch. Please find<br>
the correct version attached to this e-mail.<br>
<br>
Kind regards,<br>
<div class="im"><br>
<br>
> one of our testers found out that he could reliably reboot our<br>
> erlang-based device by just sending tons of 'A' characters to an<br>
> interactive shell.<br>
><br>
> It appears that the I/O server of the interactive shell (in group.erl)<br>
> is always reading a full line before doing any end-of-input processing,<br>
> so that by sending bytes without a newline in between, it slowly eats up<br>
> all available memory.<br>
><br>
> The patch attached to this e-mail fixes that by introducing a new<br>
> io:setopt() option in group.erl named 'max_length' that is used to check<br>
> whether the current line length exceeds this maximum whenever the line<br>
> is edited. If an overlong line is detected, max_length bytes are<br>
> returned immediately (without the actual stop condition being fulfilled,<br>
> though, allowing I/O clients to detect this situation).<br>
><br>
> max_length is allowed to be an integer() > 0 or 'unlimited'. The default<br>
> is 'unlimited' to have the old behaviour by default.<br>
><br>
><br>
><br>
> Kind regards,<br>
><br>
</div>> _______________________________________________<br>
> erlang-patches mailing list<br>
> <a href="mailto:erlang-patches@erlang.org">erlang-patches@erlang.org</a><br>
> <a href="http://erlang.org/mailman/listinfo/erlang-patches" target="_blank">http://erlang.org/mailman/listinfo/erlang-patches</a><br>
<div class="HOEnZb"><div class="h5"><br>
--<br>
Dr. Stefan Zegenhagen<br>
<br>
arcutronix GmbH<br>
Garbsener Landstr. 10<br>
30419 Hannover<br>
Germany<br>
<br>
Tel:   +49 511 277-2734<br>
Fax:   +49 511 277-2709<br>
Email: <a href="mailto:stefan.zegenhagen@arcutronix.com">stefan.zegenhagen@arcutronix.com</a><br>
Web:   <a href="http://www.arcutronix.com" target="_blank">www.arcutronix.com</a><br>
<br>
*Synchronize the Ethernet*<br>
<br>
General Managers: Dipl. Ing. Juergen Schroeder, Dr. Josef Gfrerer -<br>
Legal Form: GmbH, Registered office: Hannover, HRB 202442, Amtsgericht<br>
Hannover; Ust-Id: DE257551767.<br>
<br>
Please consider the environment before printing this message.<br>
</div></div><br>_______________________________________________<br>
erlang-patches mailing list<br>
<a href="mailto:erlang-patches@erlang.org">erlang-patches@erlang.org</a><br>
<a href="http://erlang.org/mailman/listinfo/erlang-patches" target="_blank">http://erlang.org/mailman/listinfo/erlang-patches</a><br>
<br></blockquote></div><br><br clear="all"><div><br></div>-- <br>Björn Gustavsson, Erlang/OTP, Ericsson AB
</div>