Double free in vr_message_process_response of the vrouter kernel module

Bug #1394147 reported by Sylvain Afchain
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Juniper Openstack
Fix Committed
Undecided
Sylvain Afchain
OpenContrail
Fix Committed
Undecided
Sylvain Afchain

Bug Description

the response->vr_message_buf is freed twice, one per vr_mtrans_free and per vr_message_free which causes a double free error

Tags: vrouter
Changed in opencontrail:
assignee: nobody → Sylvain Afchain (sylvain-afchain)
tags: added: vrouter
Changed in juniperopenstack:
assignee: nobody → Sylvain Afchain (safchain)
Changed in opencontrail:
status: New → In Progress
Changed in juniperopenstack:
status: New → In Progress
Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : A change has been merged

Reviewed: https://review.opencontrail.org/4752
Committed: http://github.org/Juniper/contrail-vrouter/commit/9f7b8ca9a56af6cbd84bd5f19e035b24eb83b1f8
Submitter: Zuul
Branch: master

commit 9f7b8ca9a56af6cbd84bd5f19e035b24eb83b1f8
Author: Sylvain Afchain <email address hidden>
Date: Mon Nov 17 15:44:02 2014 +0100

Fix a double free error in vr_message_process_response

This patch fixes a double free error and introduces
user land unit test for the vrouter code. The cmocka
unit test framework is used for that purpose.

A first unit test is provided that check this fixes.

Change-Id: Ic8c84d076d078a87adaf6d4398373c368db0361c
Closes-bug: #1394147

Changed in juniperopenstack:
status: In Progress → Fix Committed
Changed in opencontrail:
status: In Progress → Fix Committed
Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Reviewed: https://review.opencontrail.org/4869
Committed: http://github.org/Juniper/contrail-vrouter/commit/02b3e9402ebbf8dfb9f495f7de6890286bccb6ea
Submitter: Zuul
Branch: R2.0

commit 02b3e9402ebbf8dfb9f495f7de6890286bccb6ea
Author: Sylvain Afchain <email address hidden>
Date: Mon Nov 17 15:44:02 2014 +0100

Fix a double free error in vr_message_process_response

Change-Id: Ic8c84d076d078a87adaf6d4398373c368db0361c
Closes-bug: #1394147
(cherry picked from commit 9f7b8ca9a56af6cbd84bd5f19e035b24eb83b1f8)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Reviewed: https://review.opencontrail.org/4868
Committed: http://github.org/Juniper/contrail-vrouter/commit/e8a02ba1ac73470848df405438402d98bbfcef25
Submitter: Zuul
Branch: R1.10

commit e8a02ba1ac73470848df405438402d98bbfcef25
Author: Sylvain Afchain <email address hidden>
Date: Mon Nov 17 15:44:02 2014 +0100

Fix a double free error in vr_message_process_response

Change-Id: Ic8c84d076d078a87adaf6d4398373c368db0361c
Closes-bug: #1394147
(cherry picked from commit 9f7b8ca9a56af6cbd84bd5f19e035b24eb83b1f8)

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.