commit 64a8b9eed8f11a79602530096218cd07ab5194bd
author alshabib <alshabibi.ali@gmail.com> Wed Mar 02 17:21:37 2016 -0800
committer Gerrit Code Review <gerrit@onlab.us> Thu Mar 03 01:55:30 2016 +0000
tree 72d948720d676657708b1b3937e9b19dc88e7d7c
parent 98de57aa776d72ac9ef948e9594959a3e54b4db9

fix igmp deserialization issue

Change-Id: I38c37a6114957414ba4e13a690cd478b1116ef2c

utils/misc/src/main/java/org/onlab/packet/IGMPMembership.java

diff --git a/utils/misc/src/main/java/org/onlab/packet/IGMPMembership.java b/utils/misc/src/main/java/org/onlab/packet/IGMPMembership.java
index d5eb2d8..b44aa09 100644
--- a/utils/misc/src/main/java/org/onlab/packet/IGMPMembership.java
+++ b/utils/misc/src/main/java/org/onlab/packet/IGMPMembership.java
@@ -102,9 +102,10 @@
 
         gaddr = Ip4Address.valueOf(bb.getInt());
 
-        // Make sure we have enough buffer to hold all of these sources
-        checkBufferLength(bb.remaining(), 0, Ip4Address.BYTE_LENGTH * nsrcs);
+
         for (; nsrcs > 0; nsrcs--) {
+            // Make sure we have enough buffer to hold all of these sources
+            checkBufferLength(bb.remaining(), 0, Ip4Address.BYTE_LENGTH * nsrcs);
             Ip4Address src = Ip4Address.valueOf(bb.getInt());
             this.sources.add(src);
         }