X-Git-Url: http://git.asbjorn.biz/?a=blobdiff_plain;f=src%2Fsort.c;h=40ecad4e8514b4df1a41676f05ed17956f180143;hb=10488b0a040d2702273a7922c8b6bcebfb3cd85f;hp=6c5ae58d86cedfde993841b081a17ab71a23f68d;hpb=74aef1c34e31699595b4b198bcde5ac1af694260;p=debian%2Fdhcpd-pools.git

diff --git a/src/sort.c b/src/sort.c
index 6c5ae58..40ecad4 100644
--- a/src/sort.c
+++ b/src/sort.c
@@ -23,6 +23,7 @@
 #include <stdlib.h>
 #include <stdio.h>
 #include <string.h>
+#include <err.h>
 
 #include "dhcpd-pools.h"
 #include "defaults.h"
@@ -116,9 +117,9 @@ void field_selector(char c)
 		returner = ret_tcperc;
 		break;
 	default:
-		eprintf("field_selector: unknown sort order: %c",
-			config.sort[0]);
-		usage(EXIT_FAILURE);
+		errx(EXIT_FAILURE,
+		     "field_selector: unknown sort order `%c'",
+		     config.sort[0]);
 	}
 }
 
@@ -135,7 +136,7 @@ int get_order(struct range_t *left, struct range_t *right)
 			ret =
 			    strcmp(left->shared_net->name,
 				   right->shared_net->name);
-			if (ret > 0) {
+			if (0 < ret) {
 				return (0);
 			} else if (ret < 0) {
 				return (1);
@@ -170,7 +171,7 @@ void mergesort_ranges(struct range_t *orig, int size, struct range_t *temp)
 	if (size < MIN_MERGE_SIZE) {
 		for (left = 0; left < size; left++) {
 			hold = *(orig + left);
-			for (right = left - 1; right >= 0; right--) {
+			for (right = left - 1; 0 <= right; right--) {
 				if (get_order((orig + right), &hold)) {
 					break;
 				}