fix: off-by-one bounds check

Correct an off-by-one bounds check in `parseParameter` (`builder/vmware/common/driver_parser.go`). The condition previously used `idxAddress+1 > len(val.operand)`, which could miss the single-operand case and risk an out-of-range access.

Changed to `idxAddress+1 >= len(val.operand)` so a single address is correctly parsed as a p`ParameterRange4` with `min == max`.

Signed-off-by: Ryan Johnson <ryan.johnson@broadcom.com>

Author: tenthirtyam

builder/vmware/common/driver_parser.go

@@ -681,7 +681,7 @@ func parseParameter(val tkParameter) (pParameter, error) {
 			return nil, fmt.Errorf("invalid number of parameters for pParameterRange : %v", val.operand)
 		}
 
-		if idxAddress+1 > len(val.operand) {
+		if idxAddress+1 >= len(val.operand) {
 			res := net.ParseIP(val.operand[idxAddress])
 			return pParameterRange4{min: res, max: res}, nil
 		}
@@ -2337,7 +2337,7 @@ func readDhcpdLeaseEntry(in chan byte) (entry *dhcpLeaseEntry, err error) {
 		// means that we should probably bail.
 		return &dhcpLeaseEntry{address: matches[1]}, fmt.Errorf("missing parameters for lease entry %v", matches[1])
 
-	} else if !ok {
+	} else {
 		// If our channel is closed, so we bail "cleanly".
 		return nil, nil
 	}