diff options
| author |   Marco Varlese <marco.varlese@suse.com> | 2018-02-23 17:43:06 +0100 |
|---|---|---|
| committer |   Florin Coras <florin.coras@gmail.com> | 2018-02-25 19:33:48 +0000 |
| commit | 04e5d64c454ec53103fa1f4b7f3634bb61a65d0f (patch) | |
| tree | eb934071bb2254bea39bca2a9804caa07393b4d9 /test/test_sctp.py | |
| parent | 3473e4938718a820b63edaeab5ae7738c31379d5 (diff) | |
SCTP: fix connection memory corruption
A bug was found when multiple SCTP connections were being opened to the
same SCTP server. This patch addresses that problem, removing the use of
the 'parent' pointer approach for sub-connection and saving instead
within the sub-connection itself the ID representing its position. That
facilitates pointer-arithmetic to be computed in the
get_connection_from_transport().
Change-Id: Iaa1f4efc501590be1c93e42fd6fe3d6e02f635eb
Signed-off-by: Marco Varlese <marco.varlese@suse.com>
Diffstat (limited to 'test/test_sctp.py')
| -rw-r--r-- | test/test_sctp.py | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/test/test_sctp.py b/test/test_sctp.py index a2f5e6a45da..32068abf76e 100644 --- a/test/test_sctp.py +++ b/test/test_sctp.py @@ -68,9 +68,10 @@ class TestSCTP(VppTestCase): self.logger.critical(error) self.assertEqual(error.find("failed"), -1) - error = self.vapi.cli("test echo client mbytes 10 appns 1" + + error = self.vapi.cli("test echo client nclients 2 mbytes 100" + + " appns 1" + " fifo-size 4" + - " no-output test-bytes syn-timeout 20 " + + " no-output test-bytes syn-timeout 3" + " uri " + uri) if error: self.logger.critical(error) |