From 151f6f947451eb8b4e3893c927e05dad96b51244 Mon Sep 17 00:00:00 2001
From: Tudor Brindus <me@tbrindus.ca>
Date: Fri, 20 Dec 2019 23:28:42 -0500
Subject: [PATCH] Use PATH_MAX instead of hardcoding 255

---
 cava.c         | 2 +-
 config.c       | 2 +-
 config.h       | 3 ++-
 input/common.c | 2 ++
 4 files changed, 6 insertions(+), 3 deletions(-)

diff --git a/cava.c b/cava.c
index f9e932b..0040962 100644
--- a/cava.c
+++ b/cava.c
@@ -244,7 +244,7 @@ int main(int argc, char **argv) {
     double k[200];
     float g;
     struct timespec req = {.tv_sec = 0, .tv_nsec = 0};
-    char configPath[255];
+    char configPath[PATH_MAX];
     char *usage = "\n\
 Usage : " PACKAGE " [options]\n\
 Visualize audio input in terminal. \n\
diff --git a/config.c b/config.c
index d140450..aaae25c 100644
--- a/config.c
+++ b/config.c
@@ -341,7 +341,7 @@ bool load_colors(struct config_params *p, dictionary *ini, void *err) {
     return true;
 }
 
-bool load_config(char configPath[255], struct config_params *p, bool colorsOnly,
+bool load_config(char configPath[PATH_MAX], struct config_params *p, bool colorsOnly,
                  struct error_s *error) {
     FILE *fp;
 
diff --git a/config.h b/config.h
index 6ef108f..b34ae81 100644
--- a/config.h
+++ b/config.h
@@ -1,5 +1,6 @@
 #pragma once
 
+#include <limits.h>
 #include <stdbool.h>
 #include <stdio.h>
 #include <string.h>
@@ -66,5 +67,5 @@ struct error_s {
     int length;
 };
 
-bool load_config(char configPath[255], struct config_params *p, bool colorsOnly,
+bool load_config(char configPath[PATH_MAX], struct config_params *p, bool colorsOnly,
                  struct error_s *error);
diff --git a/input/common.c b/input/common.c
index 95561e9..3ca524f 100644
--- a/input/common.c
+++ b/input/common.c
@@ -1,5 +1,7 @@
 #include "input/common.h"
 
+#include <string.h>
+
 void reset_output_buffers(struct audio_data *data) {
     memset(data->audio_out_bass_r, 0, sizeof(int16_t) * data->FFTbassbufferSize);
     memset(data->audio_out_bass_l, 0, sizeof(int16_t) * data->FFTbassbufferSize);