Code Review / vpp.git / commit
? search:
summary | shortlog | log | commit | commitdiff | review | tree
(parent: 00b2d74) | patch
devices: vhoost cpu->copy array overflow on tcp jumbo frame (65535 bytes) 65/22965/1
authorSteven Luong <sluong@cisco.com>
Wed, 23 Oct 2019 20:28:37 +0000 (13:28 -0700)
committerSteven Luong <sluong@cisco.com>
Wed, 23 Oct 2019 20:39:18 +0000 (13:39 -0700)
commit7331005c16d0e1499080899b2a0676a0cd945595
tree2ddb0ad8c961ca3e9bebefbe18e0db3f4402018etree | snapshot
parent00b2d74d1f58b9357e8d955ad7410fb608490904commit | diff
devices: vhoost cpu->copy array overflow on tcp jumbo frame (65535 bytes)

We reserve 40 slots in cpu->copy array prior to copy out to avoid overflowing
the array. However, 40 is not enough for the jumbo frame because desceiptor
buffer len is likely at 1536. Change the reserve to 200 and add ASSERT to avoid
encountering the same problem in the future.

Type: fix

Signed-off-by: Steven Luong <sluong@cisco.com>
Change-Id: Ibf0c03c4b4f33e781d5be8679ccd6c3a4b4a646d
src/vnet/devices/virtio/vhost_user_input.c diff | blob | history
src/vnet/devices/virtio/vhost_user_output.c diff | blob | history
Vector Packet Processing