fix: return error strings instead of NULL on popen and realloc failures in tool_function_bash

Browse Source
This commit is contained in:
retoor 2025-03-03 08:48:50 +00:00
parent f8667d1a4b
commit 00c45e0978
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
tools.h
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 NULL; return "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 NULL; return "Failed to allocate memory!";
} }
output = new_output; output = new_output;
strcpy(output + total_size, buffer); strcpy(output + total_size, buffer);