From 21f045ac7eb2ecdd3bab919c9fc9ebb7dde84dd7 Mon Sep 17 00:00:00 2001 From: Mehak Mahajan Date: Wed, 15 Aug 2012 11:19:35 -0700 Subject: [PATCH] Correct number of bytes to allocated for slaves in bundle action. The size of each slave is a uint16_t. This means that each slave needs 2 bytes at the end of nx_action_bundle. Earlier, the size of each slave was not being factored in when allocating space. This commit corrects that by allocating 2 bytes for each slave when calculating the total number of bytes to be allocated at the end of nx_action_bundle. Signed-off-by: Mehak Mahajan --- lib/bundle.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/bundle.c b/lib/bundle.c index c6b1f75eb..e0f8e6b6c 100644 --- a/lib/bundle.c +++ b/lib/bundle.c @@ -217,7 +217,7 @@ bundle_check(const struct ofpact_bundle *bundle, int max_ports, void bundle_to_nxast(const struct ofpact_bundle *bundle, struct ofpbuf *openflow) { - int slaves_len = ROUND_UP(bundle->n_slaves, OFP_ACTION_ALIGN); + int slaves_len = ROUND_UP(2 * bundle->n_slaves, OFP_ACTION_ALIGN); struct nx_action_bundle *nab; ovs_be16 *slaves; size_t i; -- 2.43.0