Overload and non-parsed status processing is failing

[desertwitch]

OVER and non-processed statuses never get evaluated as there is a bug in the LB status evaluation within this function:

wmnut/src/wmnut.c

Lines 136 to 150 in e3255f7

	if (get_ups_var ("ups.status", value) > NOK) {
	if (!strncmp(value, "OL", 2))
	CurHost->ups_status = UPS_ONLINE;
	else if (!strncmp(value, "OB", 2))
	CurHost->ups_status = UPS_ONBATT;
	else if (!strstr(value, "LB"))
	CurHost->ups_status = UPS_LOWBATT;
	else if (!strncmp(value, "OVER", 4))
	CurHost->ups_status = UPS_OVERLOAD;
	else
	CurHost->ups_status = UPS_DEFAULT;
	}
	else
	CurHost->ups_status = UPS_DEFAULT;

This else if checks if LB is not contained and sets the status to UPS_LOWBATT, but should check if it is contained here:

wmnut/src/wmnut.c

Line 142 in e3255f7

else if (!strstr(value, "LB"))

I'm overhauling this function as part of a larger pull request in the works (#8), so this issue is mostly a reminder to myself.