Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1945)

Unified Diff: net/third_party/nss/ssl/sslsecur.c

Issue 9764001: Add DTLS support to NSS, contributed by Eric Rescorla. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Made one pass, dtls1con.c not reviewed yet Created 8 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: net/third_party/nss/ssl/sslsecur.c
===================================================================
--- net/third_party/nss/ssl/sslsecur.c (revision 127709)
+++ net/third_party/nss/ssl/sslsecur.c (working copy)
@@ -615,6 +615,7 @@
if (!(flags & PR_MSG_PEEK)) {
ss->gs.readOffset += amount;
}
+ PORT_Assert(ss->gs.readOffset <= ss->gs.writeOffset);
wtc 2012/03/21 23:12:16 Nit: should we assert the invariant before the ret
ekr 2012/03/22 15:07:17 I'm not sure about the first question, but your la
rv = amount;
SSL_TRC(30, ("%d: SSL[%d]: amount=%d available=%d",

Powered by Google App Engine
This is Rietveld 408576698