diff --git a/src/energy.c b/src/energy.c
index a5a7c777..324ec80e 100644
--- a/src/energy.c
+++ b/src/energy.c
@@ -96,12 +96,12 @@ void calculate_pocket_energy_grids(c_lst_pockets *pockets, s_fparams *params, s_
 
 
 
-        sprintf(final_path, "%s/pockets/pocket_%d_vdw.dx", out_path, pocket_number);
+        snprintf(final_path, sizeof(final_path), "%s/pockets/pocket_%d_vdw.dx", out_path, pocket_number);
         f_vdw = fopen(final_path, "w");
         write_grid(pocket_vdw_grid, f_vdw);
         fclose(f_vdw);
 
-        sprintf(final_path, "%s/pockets/pocket_%d_elec.dx", out_path, pocket_number);
+        snprintf(final_path, sizeof(final_path), "%s/pockets/pocket_%d_elec.dx", out_path, pocket_number);
         f_elec = fopen(final_path, "w");
         write_grid(pocket_elec_grid, f_elec);
         fclose(f_elec);
diff --git a/src/fparams.c b/src/fparams.c
index feea5100..8ee306f8 100644
--- a/src/fparams.c
+++ b/src/fparams.c
@@ -122,7 +122,7 @@ s_fparams *get_fpocket_args(int nargs, char **args)
     opterr = 0;
     char *pt;
     char *apt;
-    char *residue_string[M_MAX_CUSTOM_POCKET_LEN];
+    char residue_string[M_MAX_CUSTOM_POCKET_LEN];
     short custom_ligand_i = 0;
     const char *separators = ",:";
     static struct option fplong_options[] = {/*long options args located in fparams.h*/
@@ -293,7 +293,7 @@ s_fparams *get_fpocket_args(int nargs, char **args)
             rest = par->custom_pocket_arg;
             while ((pt = strtok_r(rest, ".", &rest)))
             {
-                strcpy(&residue_string, pt);
+                strcpy(residue_string, pt);
                 rest2 = residue_string;
                 apti = 0;
                 while ((apt = strtok_r(rest2, ":", &rest2)))
diff --git a/src/mdparams.c b/src/mdparams.c
index ea50170e..f671dc4a 100644
--- a/src/mdparams.c
+++ b/src/mdparams.c
@@ -132,7 +132,7 @@ s_mdparams* get_mdpocket_args(int nargs, char **args) {
     char *str_list_file = NULL;
     char **args_copy = my_malloc(sizeof (char**) *nargs);
     for (i = 0; i < nargs; i++) {
-        args_copy[i] = my_malloc(sizeof (args[i]));
+        args_copy[i] = my_malloc(strlen(args[i]) + 1);
         strcpy(args_copy[i], args[i]);
     }