Comment 8 for bug 1446550

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

commit e937aa9b4cace0eac361439403af5fa01355bfe2
Author: Anand H. Krishnan <email address hidden>
Date: Mon Apr 20 12:41:11 2015 +0530

Free the defer data in case of errors

To make sure that we flush all the packets that are queued in a flow
entry, we run a defer function. If for any reason this defer was not
scheduled (because the function was called with no hold queue), the
defer data has to be freed.

Closes-BUG: #1436798
(cherry picked from commit 8c30ce9a3e254d45dc4de595cc066f2da21c18d6)

Fix improper boundary checks and reference count leaks

Boundary checks allow for one extra label than the maximum, causing
memory corruption. Also, when a label is changed, reference to old
nexthop has to be released. Two harmless boundary checks in nexthop
subysystem is also addressed.

Closes-BUG: #1446550
Change-Id: I9289265b8a843160fdfe6fffc3e52c131d9b2a4a