s2n_recv: upon connection close, *more needs to be set to zero.

[samrushing]

If *more is left set to 1, there's no indication to the caller that the connection has closed.
Additionally, the wipe of the connection structure sets read/writefd to zero, so the end result is another call to s2n_recv() that is reading from stdin.

The 3 lines of changes are in my fork, let me know if you want me to do a pull request.

There may be additional such issues with *more, e.g. s2n_flush().

Note: this problem showed up while testing a server in non-blocking mode.

[colmmacc]

More than happy to fix, and ecstatic to hear about a non-blocking use-case. Please do submit a PR. Should be easy to integrate.

[hyandell]

Ready to resolve Colm, or did you want to keep open to look into s2n_flush and others?

[alexw91]

This looks like it has been fixed with #166 if not, please reopen issue.