diff options
author | Azat Khuzhin <a3at.mail@gmail.com> | 2018-11-14 00:20:20 +0300 |
---|---|---|
committer | Azat Khuzhin <a3at.mail@gmail.com> | 2018-11-20 06:22:31 +0300 |
commit | b98d32d0c90864ff60bc689f3bcf738af58ca79c (patch) | |
tree | 0fc220aab9ec46e497d76c7eb6e7042c7e00ef93 /bufferevent.c | |
parent | 3036f15a176b038f42d3896463c39058c6bac943 (diff) | |
download | libevent-b98d32d0c90864ff60bc689f3bcf738af58ca79c.tar.gz |
http: improve error path for bufferevent_{setfd,enable,disable}()
We have calls to the next functions but do not check return values,
though they can be invalid and it is better to show this somehow.
Also do bufferevent_setfd() first and only after it
bufferevent_enable()/bufferevent_disable() since:
a) it is more natural
b) it will avoid extra operations
c) it will not fail first bufferevent_enable() (this is the case for
buffbufferevent_async at least)
In this case we could add more information for issues like #709
Diffstat (limited to 'bufferevent.c')
0 files changed, 0 insertions, 0 deletions