Merge pull request #54 from troglobit/brillo-merges

Brillo merges

src/light.c

 			ctx.field = LIGHT_MIN_CAP;
 			break;
 
-			/* -- Controller selection -- */
 		case 'a':
 			ASSERT_CTRLSET();
 			ctx.ctrl = LIGHT_AUTO;
 			ctx.ctrl = LIGHT_SPECIFY;
 			if (!light_check_ctrl(optarg))
 				return false;
-			strncpy(ctx.ctrl_name, optarg, NAME_MAX);
-			ctx.ctrl_name[NAME_MAX] = '\0';
+
+			snprintf(ctx.ctrl_name, sizeof(ctx.ctrl_name), "%s", optarg);
 			break;
-			/* -- Value modes -- */
 
 		case 'p':
 			ASSERT_VALSET();
 	/* Make sure we have a valid controller before we proceed */
 	if (ctx.ctrl == LIGHT_AUTO) {
 		LIGHT_NOTE("Automatic mode -- finding best controller");
-		if (!light_ctrl_probe(ctx.ctrl_name)) {
+		if (!light_ctrl_probe(ctx.ctrl_name, sizeof(ctx.ctrl_name))) {
 			LIGHT_ERR("could not find suitable controller");
 			return false;
 		}
 	return true;
 }
 
-static bool light_ctrl_iterate(DIR *dir, char *current)
+static bool light_ctrl_iterate(DIR *dir, char *current, size_t len)
 {
 	struct dirent *d;
 	bool found = false;
 		}
 	}
 
-	strncpy(current, d->d_name, NAME_MAX);
-	current[NAME_MAX] = '\0';
+	snprintf(current, len, "%s", d->d_name);
 
 	return true;
 }
 
 /* WARNING: `controller` HAS to be at most NAME_MAX, otherwise fails */
-bool light_ctrl_probe(char *controller)
+bool light_ctrl_probe(char *controller, size_t len)
 {
 	DIR *dir;
 	unsigned long best = 0;
 		return false;
 	}
 
-	while (light_ctrl_iterate(dir, current)) {
+	while (light_ctrl_iterate(dir, current, sizeof(current))) {
 		unsigned long val = 0;
 
 		LIGHT_NOTE("found '%s' controller", current);
 				if (val > best) {
 					found = true;
 					best = val;
-					strncpy(best_name, current, NAME_MAX);
-					best_name[NAME_MAX] = '\0';
+					snprintf(best_name, sizeof(best_name), "%s", current);
 					ctx.has_cached_brightness_max = true;
 					ctx.cached_brightness_max = val;
 				} else {
 		return false;
 	}
 
-	strncpy(controller, best_name, NAME_MAX);
-	controller[NAME_MAX] = '\0';
+	snprintf(controller, len, "%s", best_name);
+
 	return true;
 }
 
 
 bool light_ctrl_list(void)
 {
-	char controller[NAME_MAX + 1];
+	char controller[NAME_MAX];
 	bool found = false;
 	DIR *dir;
 
 		return false;
 	}
 
-	while (light_ctrl_iterate(dir, controller)) {
+	while (light_ctrl_iterate(dir, controller, sizeof(controller))) {
 		printf("%s\n", controller);
 		found = true;
 	}

src/light.h

 
 typedef struct {
 	/* Cache file prefix */
-	char               prefix[NAME_MAX + 1];
+	char               prefix[NAME_MAX];
 
 	/* Which controller to use */
 	light_ctrl_mode_t  ctrl;
 void light_free                    (void);
 
 bool light_ctrl_list               (void);
-bool light_ctrl_probe              (char *controller);
+bool light_ctrl_probe              (char *controller, size_t len);
 bool light_ctrl_exist              (char const *controller);
 
 bool light_ctrl_get_brightness     (char const *controller, unsigned long *v);