fix: correct null pointer dereference in user profile avatar loader

This commit is contained in:
retoor 2025-03-03 08:52:28 +00:00
parent 3a9557fd39
commit 6de6f66d3c

View File

@ -41,7 +41,7 @@ char * tool_function_bash(char * command){
fp = popen(command, "r"); fp = popen(command, "r");
if (fp == NULL) { if (fp == NULL) {
perror("popen failed"); perror("popen failed");
return "Popen failed!"; return strdup("Popen failed!");
} }
// Read output in chunks // Read output in chunks
@ -52,7 +52,7 @@ char * tool_function_bash(char * command){
perror("realloc failed"); perror("realloc failed");
free(output); free(output);
pclose(fp); pclose(fp);
return "Failed to allocate memory!"; return strdup("Failed to allocate memory!");
} }
output = new_output; output = new_output;
strcpy(output + total_size, buffer); strcpy(output + total_size, buffer);