commit af19de31018421c3deea655dacca64211a6eb327
author David Benjamin <davidben@chromium.org> Sun Jan 11 19:50:31 2015 -0500
committer Adam Langley <agl@google.com> Wed Jan 14 21:48:40 2015 +0000
tree 9c992ec0db1245b06c7dd88ed21eb70e141b7c88
parent 17a5f85cbbb8e637af0abfccfd8f21eb1054c9a3

Fix the test async_bio in datagram mode.

write_quota should only be decremented by 1 in datagram mode, otherwise we'll
underflow and always allow writes through. This does not cause any existing
tests to fail.

(It will be useful once the bug in dtls1_do_write is fixed.)

Change-Id: I42aa001d7264790a3726269890635f679497fb1c
Reviewed-on: https://boringssl-review.googlesource.com/2831
Reviewed-by: Adam Langley <agl@google.com>

ssl/test/async_bio.cc

diff --git a/ssl/test/async_bio.cc b/ssl/test/async_bio.cc
index 4861579..88f8e9a 100644
--- a/ssl/test/async_bio.cc
+++ b/ssl/test/async_bio.cc
@@ -61,7 +61,7 @@
   if (ret <= 0) {
     BIO_copy_next_retry(bio);
   } else {
-    a->write_quota -= ret;
+    a->write_quota -= (a->datagram ? 1 : ret);
   }
   return ret;
 }