aboutsummaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
authormilo <milo@r0ot.me>2011-02-09 01:07:30 +0100
committermilo <milo@r0ot.me>2011-02-10 14:35:29 +0100
commit3eaf25d9893822e021642d68bfcd6143571800d8 (patch)
tree2ccdaac25338e52d136ae24333707db4e04eb95a /src
parent6f2bf91e29e90758f71d5d5bf5742c225bdd5fbf (diff)
downloadlibssh-3eaf25d9893822e021642d68bfcd6143571800d8.tar.gz
libssh-3eaf25d9893822e021642d68bfcd6143571800d8.tar.xz
libssh-3eaf25d9893822e021642d68bfcd6143571800d8.zip
Fix NULL pointer checks in poll
Diffstat (limited to 'src')
-rw-r--r--src/poll.c15
1 files changed, 7 insertions, 8 deletions
diff --git a/src/poll.c b/src/poll.c
index ab2fcc47..156457cc 100644
--- a/src/poll.c
+++ b/src/poll.c
@@ -300,6 +300,9 @@ int ssh_poll(ssh_pollfd_t *fds, nfds_t nfds, int timeout) {
* @param events Poll events that will be monitored for the socket. i.e.
* POLLIN, POLLPRI, POLLOUT, POLLERR, POLLHUP, POLLNVAL
* @param cb Function to be called if any of the events are set.
+ * The prototype of cb is:
+ * int (*ssh_poll_callback)(ssh_poll_handle p, socket_t fd,
+ * int revents, void *userdata);
* @param userdata Userdata to be passed to the callback function. NULL if
* not needed.
*
@@ -318,12 +321,8 @@ ssh_poll_handle ssh_poll_new(socket_t fd, short events, ssh_poll_callback cb,
p->x.fd = fd;
p->events = events;
- if (cb != NULL) {
- p->cb = cb;
- }
- if (userdata != NULL) {
- p->cb_data = userdata;
- }
+ p->cb = cb;
+ p->cb_data = userdata;
return p;
}
@@ -490,7 +489,7 @@ void ssh_poll_ctx_free(ssh_poll_ctx ctx) {
socket_t fd = ctx->pollfds[i].fd;
/* force poll object removal */
- if (p->cb(p, fd, POLLERR, p->cb_data) < 0) {
+ if (p->cb && p->cb(p, fd, POLLERR, p->cb_data) < 0) {
used = ctx->polls_used;
} else {
i++;
@@ -648,7 +647,7 @@ int ssh_poll_ctx_dopoll(ssh_poll_ctx ctx, int timeout) {
fd = ctx->pollfds[i].fd;
revents = ctx->pollfds[i].revents;
- if (p->cb(p, fd, revents, p->cb_data) < 0) {
+ if (p->cb && p->cb(p, fd, revents, p->cb_data) < 0) {
/* the poll was removed, reload the used counter and start again */
used = ctx->polls_used;
i=0;