Build 2715: Stack corruption happened when config was done to mirror a port of analyzer to the analyzer itself

[Ankit Jain]

Steps : Do the following config:

1) Port : Analyzer Port:
423f53b1-7b08-424d-b829-dd36f548a338
vn1
10.1.1.8
-compute:nova

2) VM: Analyzer3:

admin nodei4
analyzer3
 analyzer

    10.1.1.8

 m1.medium Active None Running 4 hours, 12 minutes

3) Mirroring enabled on port 10.1.1.8 with the following details:

analyzer ip : 10.1.1.8
Analyzer Name : default-domain:admin:analyzer3
udp port: 8099
routing instance : vn1
Direction : Both

With the above config, stack corruption was seen on nodei4

Anand had debugged the issue.

[OpenContrail Admin (ci-admin-f)]

Review in progress for https://review.opencontrail.org/17883
Submitter: Divakar Dharanalakota (<email address hidden>)

[OpenContrail Admin (ci-admin-f)]

Reviewed: https://review.opencontrail.org/17883
Committed: http://github.org/Juniper/contrail-vrouter/commit/c1c472caf7209d3fd117d3d58c4d42dab7210505
Submitter: Zuul
Branch: R3.0

commit c1c472caf7209d3fd117d3d58c4d42dab7210505
Author: Divakar <email address hidden>
Date: Fri Feb 26 11:21:58 2016 +0530

Dont mirror the same packet again

If mirroring is enabled on analyzing VM's port itself, one copy of the
original packet gets mirrored to that port. But this mirrored packet
should not be mirroed again. Right now without verifying whether the
packet is mirrored or not, it is attempted for mirroring again.

When a packet is mirrored it is marked with flag VP_FLAG_FROM_DP. If
this flag is set it is not mirrored again.

Change-Id: Ie4cca916945878b77bc4db7666992f3933db3e13
closes-bug: #1549761

[OpenContrail Admin (ci-admin-f)]

Review in progress for https://review.opencontrail.org/17992
Submitter: Divakar Dharanalakota (<email address hidden>)

[OpenContrail Admin (ci-admin-f)]

Reviewed: https://review.opencontrail.org/17992
Committed: http://github.org/Juniper/contrail-vrouter/commit/0abcc4dbc42246e148ba74ef7d85825738fa0ece
Submitter: Zuul
Branch: master

commit 0abcc4dbc42246e148ba74ef7d85825738fa0ece
Author: Divakar <email address hidden>
Date: Fri Feb 26 11:21:58 2016 +0530

Dont mirror the same packet again

If mirroring is enabled on analyzing VM's port itself, one copy of the
original packet gets mirrored to that port. But this mirrored packet
should not be mirroed again. Right now without verifying whether the
packet is mirrored or not, it is attempted for mirroring again.

When a packet is mirrored it is marked with flag VP_FLAG_FROM_DP. If
this flag is set it is not mirrored again.

Change-Id: Ie4cca916945878b77bc4db7666992f3933db3e13
closes-bug: #1549761

[OpenContrail Admin (ci-admin-f)]

Review in progress for https://review.opencontrail.org/18464
Submitter: Divakar Dharanalakota (<email address hidden>)

[OpenContrail Admin (ci-admin-f)]

Reviewed: https://review.opencontrail.org/18464
Committed: http://github.org/Juniper/contrail-vrouter/commit/c6fcf12d4b47416ee97fa44fd15aa4a81a005e67
Submitter: Zuul
Branch: R2.21.x

commit c6fcf12d4b47416ee97fa44fd15aa4a81a005e67
Author: Divakar <email address hidden>
Date: Fri Feb 26 11:21:58 2016 +0530

Dont mirror the same packet again

If mirroring is enabled on analyzing VM's port itself, one copy of the
original packet gets mirrored to that port. But this mirrored packet
should not be mirroed again. Right now without verifying whether the
packet is mirrored or not, it is attempted for mirroring again.

When a packet is mirrored it is marked with flag VP_FLAG_FROM_DP. If
this flag is set it is not mirrored again.

Change-Id: Ie4cca916945878b77bc4db7666992f3933db3e13
closes-bug: #1549761