diff options
author | Sree Harsha Totakura <totakura@in.tum.de> | 2012-09-30 15:11:39 +0000 |
---|---|---|
committer | Sree Harsha Totakura <totakura@in.tum.de> | 2012-09-30 15:11:39 +0000 |
commit | 0c1c5a9be5e6cd117736a569e60ae79c739c56ef (patch) | |
tree | c47e04af1bc3f6470b62fcd02718712b0ef9316f /src/stream | |
parent | 2a6f6cb8fd439dd39e0e222948c2ed792a4f1132 (diff) | |
download | gnunet-0c1c5a9be5e6cd117736a569e60ae79c739c56ef.tar.gz gnunet-0c1c5a9be5e6cd117736a569e60ae79c739c56ef.zip |
more fixes for #2570
Diffstat (limited to 'src/stream')
-rw-r--r-- | src/stream/perf_stream_api.c | 9 | ||||
-rw-r--r-- | src/stream/test_stream_2peers.c | 9 | ||||
-rw-r--r-- | src/stream/test_stream_2peers_halfclose.c | 12 | ||||
-rw-r--r-- | src/stream/test_stream_big.c | 8 | ||||
-rw-r--r-- | src/stream/test_stream_local.c | 8 | ||||
-rw-r--r-- | src/stream/test_stream_sequence_wraparound.c | 8 |
6 files changed, 49 insertions, 5 deletions
diff --git a/src/stream/perf_stream_api.c b/src/stream/perf_stream_api.c index 3dbdda256..e7e37792a 100644 --- a/src/stream/perf_stream_api.c +++ b/src/stream/perf_stream_api.c | |||
@@ -588,6 +588,15 @@ stream_listen_cb (void *cls, struct GNUNET_STREAM_Socket *socket, | |||
588 | { | 588 | { |
589 | struct PeerData *pdata = cls; | 589 | struct PeerData *pdata = cls; |
590 | 590 | ||
591 | |||
592 | if ((NULL == socket) || (NULL == initiator)) | ||
593 | { | ||
594 | GNUNET_log (GNUNET_ERROR_TYPE_WARNING, "Binding error\n"); | ||
595 | if (GNUNET_SCHEDULER_NO_TASK != abort_task) | ||
596 | GNUNET_SCHEDULER_cancel (abort_task); | ||
597 | abort_task = GNUNET_SCHEDULER_add_now (&do_abort, NULL); | ||
598 | return GNUNET_OK; | ||
599 | } | ||
591 | GNUNET_assert (NULL != socket); | 600 | GNUNET_assert (NULL != socket); |
592 | GNUNET_assert (pdata == &peer_data[1]); | 601 | GNUNET_assert (pdata == &peer_data[1]); |
593 | GNUNET_log (GNUNET_ERROR_TYPE_DEBUG, "Peer connected: %s\n", | 602 | GNUNET_log (GNUNET_ERROR_TYPE_DEBUG, "Peer connected: %s\n", |
diff --git a/src/stream/test_stream_2peers.c b/src/stream/test_stream_2peers.c index 86d1a84ad..c517cc2b1 100644 --- a/src/stream/test_stream_2peers.c +++ b/src/stream/test_stream_2peers.c | |||
@@ -427,7 +427,14 @@ stream_listen_cb (void *cls, | |||
427 | struct GNUNET_STREAM_Socket *socket, | 427 | struct GNUNET_STREAM_Socket *socket, |
428 | const struct GNUNET_PeerIdentity *initiator) | 428 | const struct GNUNET_PeerIdentity *initiator) |
429 | { | 429 | { |
430 | GNUNET_assert (NULL != socket); | 430 | if ((NULL == socket) || (NULL == initiator)) |
431 | { | ||
432 | GNUNET_log (GNUNET_ERROR_TYPE_WARNING, "Binding error\n"); | ||
433 | if (GNUNET_SCHEDULER_NO_TASK != abort_task) | ||
434 | GNUNET_SCHEDULER_cancel (abort_task); | ||
435 | abort_task = GNUNET_SCHEDULER_add_now (&do_abort, NULL); | ||
436 | return GNUNET_OK; | ||
437 | } | ||
431 | GNUNET_assert (NULL != initiator); | 438 | GNUNET_assert (NULL != initiator); |
432 | GNUNET_assert (socket != peer2.socket); | 439 | GNUNET_assert (socket != peer2.socket); |
433 | GNUNET_assert (0 == memcmp (initiator, &peer2.our_id, | 440 | GNUNET_assert (0 == memcmp (initiator, &peer2.our_id, |
diff --git a/src/stream/test_stream_2peers_halfclose.c b/src/stream/test_stream_2peers_halfclose.c index 2d14acfe3..2502615c2 100644 --- a/src/stream/test_stream_2peers_halfclose.c +++ b/src/stream/test_stream_2peers_halfclose.c | |||
@@ -661,15 +661,19 @@ stream_listen_cb (void *cls, | |||
661 | struct GNUNET_STREAM_Socket *socket, | 661 | struct GNUNET_STREAM_Socket *socket, |
662 | const struct GNUNET_PeerIdentity *initiator) | 662 | const struct GNUNET_PeerIdentity *initiator) |
663 | { | 663 | { |
664 | GNUNET_assert (NULL != socket); | 664 | if ((NULL == socket) || (NULL == initiator)) |
665 | GNUNET_assert (NULL != initiator); | 665 | { |
666 | GNUNET_log (GNUNET_ERROR_TYPE_WARNING, "Binding error\n"); | ||
667 | if (GNUNET_SCHEDULER_NO_TASK != abort_task) | ||
668 | GNUNET_SCHEDULER_cancel (abort_task); | ||
669 | abort_task = GNUNET_SCHEDULER_add_now (&do_abort, NULL); | ||
670 | return GNUNET_OK; | ||
671 | } | ||
666 | GNUNET_assert (socket != peer1.socket); | 672 | GNUNET_assert (socket != peer1.socket); |
667 | |||
668 | GNUNET_log (GNUNET_ERROR_TYPE_DEBUG, | 673 | GNUNET_log (GNUNET_ERROR_TYPE_DEBUG, |
669 | "%s: Peer connected: %s\n", | 674 | "%s: Peer connected: %s\n", |
670 | GNUNET_i2s (&peer2.our_id), | 675 | GNUNET_i2s (&peer2.our_id), |
671 | GNUNET_i2s(initiator)); | 676 | GNUNET_i2s(initiator)); |
672 | |||
673 | peer2.socket = socket; | 677 | peer2.socket = socket; |
674 | /* FIXME: reading should be done right now instead of a scheduled call */ | 678 | /* FIXME: reading should be done right now instead of a scheduled call */ |
675 | read_task = GNUNET_SCHEDULER_add_now (&stream_read, (void *) socket); | 679 | read_task = GNUNET_SCHEDULER_add_now (&stream_read, (void *) socket); |
diff --git a/src/stream/test_stream_big.c b/src/stream/test_stream_big.c index 19c7e571a..05da82691 100644 --- a/src/stream/test_stream_big.c +++ b/src/stream/test_stream_big.c | |||
@@ -302,6 +302,14 @@ stream_listen_cb (void *cls, | |||
302 | struct GNUNET_STREAM_Socket *socket, | 302 | struct GNUNET_STREAM_Socket *socket, |
303 | const struct GNUNET_PeerIdentity *initiator) | 303 | const struct GNUNET_PeerIdentity *initiator) |
304 | { | 304 | { |
305 | if ((NULL == socket) || (NULL == initiator)) | ||
306 | { | ||
307 | GNUNET_log (GNUNET_ERROR_TYPE_WARNING, "Binding error\n"); | ||
308 | if (GNUNET_SCHEDULER_NO_TASK != abort_task) | ||
309 | GNUNET_SCHEDULER_cancel (abort_task); | ||
310 | abort_task = GNUNET_SCHEDULER_add_now (&do_abort, NULL); | ||
311 | return GNUNET_OK; | ||
312 | } | ||
305 | GNUNET_assert (NULL != socket); | 313 | GNUNET_assert (NULL != socket); |
306 | GNUNET_assert (socket != peer1.socket); | 314 | GNUNET_assert (socket != peer1.socket); |
307 | 315 | ||
diff --git a/src/stream/test_stream_local.c b/src/stream/test_stream_local.c index 699bc8032..3286ade5f 100644 --- a/src/stream/test_stream_local.c +++ b/src/stream/test_stream_local.c | |||
@@ -320,6 +320,14 @@ stream_listen_cb (void *cls, | |||
320 | { | 320 | { |
321 | struct PeerData *peer=cls; | 321 | struct PeerData *peer=cls; |
322 | 322 | ||
323 | if ((NULL == socket) || (NULL == initiator)) | ||
324 | { | ||
325 | GNUNET_log (GNUNET_ERROR_TYPE_WARNING, "Binding error\n"); | ||
326 | if (GNUNET_SCHEDULER_NO_TASK != abort_task) | ||
327 | GNUNET_SCHEDULER_cancel (abort_task); | ||
328 | abort_task = GNUNET_SCHEDULER_add_now (&do_abort, NULL); | ||
329 | return GNUNET_OK; | ||
330 | } | ||
323 | GNUNET_assert (NULL != socket); | 331 | GNUNET_assert (NULL != socket); |
324 | GNUNET_assert (socket != peer1.socket); | 332 | GNUNET_assert (socket != peer1.socket); |
325 | GNUNET_assert (&peer2 == peer); | 333 | GNUNET_assert (&peer2 == peer); |
diff --git a/src/stream/test_stream_sequence_wraparound.c b/src/stream/test_stream_sequence_wraparound.c index 8f35a41d3..f23171661 100644 --- a/src/stream/test_stream_sequence_wraparound.c +++ b/src/stream/test_stream_sequence_wraparound.c | |||
@@ -302,6 +302,14 @@ stream_listen_cb (void *cls, | |||
302 | struct GNUNET_STREAM_Socket *socket, | 302 | struct GNUNET_STREAM_Socket *socket, |
303 | const struct GNUNET_PeerIdentity *initiator) | 303 | const struct GNUNET_PeerIdentity *initiator) |
304 | { | 304 | { |
305 | if ((NULL == socket) || (NULL == initiator)) | ||
306 | { | ||
307 | GNUNET_log (GNUNET_ERROR_TYPE_WARNING, "Binding error\n"); | ||
308 | if (GNUNET_SCHEDULER_NO_TASK != abort_task) | ||
309 | GNUNET_SCHEDULER_cancel (abort_task); | ||
310 | abort_task = GNUNET_SCHEDULER_add_now (&do_abort, NULL); | ||
311 | return GNUNET_OK; | ||
312 | } | ||
305 | GNUNET_assert (NULL != socket); | 313 | GNUNET_assert (NULL != socket); |
306 | GNUNET_assert (socket != peer1.socket); | 314 | GNUNET_assert (socket != peer1.socket); |
307 | GNUNET_log (GNUNET_ERROR_TYPE_DEBUG, | 315 | GNUNET_log (GNUNET_ERROR_TYPE_DEBUG, |