[svn-commits] mattf: branch 1.4 r1702 - /branches/1.4/q921.c

[SVN commits to the Digium repositories]

Author: mattf
Date: Wed May 19 16:30:34 2010
New Revision: 1702

URL: http://svnview.digium.com/svn/libpri?view=rev&rev=1702
Log:
It's amazing what a tiny bug in the Q.921 SDL diagram can do to cause trouble....  Fix issue where V_R was not reset and N_R was consequentially transmitted incorrectly.  Particularly in layer 2 initiated re-establishments.

Modified:
    branches/1.4/q921.c

Modified: branches/1.4/q921.c
URL: http://svnview.digium.com/svn/libpri/branches/1.4/q921.c?view=diff&rev=1702&r1=1701&r2=1702
==============================================================================
--- branches/1.4/q921.c (original)
+++ branches/1.4/q921.c Wed May 19 16:30:34 2010
@@ -1342,8 +1342,6 @@
 		} else {
 			/* Might not want this... */
 			pri->l3initiated = 0;
-			/* But do want this */
-			pri->v_r = 0;
 			/* return DL-ESTABLISH-CONFIRM */
 		}
 
@@ -1355,7 +1353,7 @@
 		stop_t200(pri);
 		start_t203(pri);
 
-		pri->v_s = pri->v_a = 0;
+		pri->v_r = pri->v_s = pri->v_a = 0;
 
 		q921_setstate(pri, Q921_MULTI_FRAME_ESTABLISHED);
 		break;