Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Disallow a new sub-interface for an existing vlan
Do not do a 'preset' on mirror packets When sub-interfaces with the same vlan id are added again with a different vif index, we overwrite the existing entry in the vlan table with the new vif. On deletion of the old vif, we check whether the pointer in the vlan table match or not. If it does not match, we do not remove the napi setup, but go ahead and free the napi structure. This could potentially result in memory corruption. To avoid such scenarios, return an error if a vif already exists for a particular vlan When a packet comes from vhost, we typically accept the layer 2 header that is supplied by the host and hence a 'preset' is done to reset the packet to what it was before it entered vRouter. Unfortunately, this was also done for packets those mirrored such packets, leading to offset issues since we do end up adding a lot more headers than what the original packets contain. This leads to wrong length calculation and hence searching for more skbs than that is originally present and a crash. Change-Id: I358c93d0d2f6c9f8bfaea1e04885b390d089baf9 Closes-BUG: #1543534, #1544832
- Loading branch information