这个非常简单的C程序有什么问题？ （导致内存泄漏）

Question

我有一个非常简单的C程序，用于每秒执行一次对php的系统调用，执行一个php脚本，该脚本将数据库中待处理的推送通知发送给Apple。但是无论如何，该程序会在大约10小时后导致内存溢出，因此我将线程创建之间的睡眠时间从1s减少到10000us，并且我可以用htop实时看到内存在不断增加。这是程序：

#include <stdlib.h>
#include <stdio.h>
#include <pthread.h>
#include <unistd.h>

typedef struct {
    char* script_path ;
} arg_for_script ;

static void *start_instance(void *_args)
{
    int id = abs(pthread_self());
    arg_for_script* args = _args ;

    printf("[SERVICE] start php script on thread %d\n",id);
    fflush(stdout);
    char cmd[200] ;

    sprintf(cmd, "php -f %s %d", args->script_path, id );

    system(cmd);

    printf("[SERVICE] end of script on thread %d\n", id);
    fflush(stdout);



    pthread_exit(NULL);
}

int main(int argc, char* argv[])
{


    if(argc < 2)
    {
        fprintf(stderr, "[SERVICE] Path of php notification script must be filled\n");
        fflush(stderr);
        return EXIT_FAILURE;
    }

    arg_for_script args ;

    args.script_path = argv[1];

    pthread_attr_t tattr ;
    struct sched_param param;
    param.sched_priority = 1 ;

    pthread_attr_init(&tattr);

    pthread_attr_setinheritsched(&tattr, PTHREAD_EXPLICIT_SCHED);
    pthread_attr_setschedpolicy(&tattr, SCHED_FIFO);
    pthread_attr_setschedparam(&tattr, &param);



    while(1) {
        pthread_t thrd;

      //  if(pthread_create(&thrd, &tattr, start_instance, (void *)&args) == -1) {
        if(pthread_create(&thrd, NULL, start_instance, (void *)&args) == -1)
        {
            fprintf(stderr, "[SERVICE] Unable to create thread\n");
            fflush(stderr);
            return EXIT_FAILURE;
        }

        usleep( 10000);
    }

  //  pthread_attr_destroy(&tattr);
    return EXIT_SUCCESS ;

}

我对C有所了解，我记得我需要使用malloc释放自己仅分配的内存。这里我不做这种动态内存分配。那么，为什么该程序会增加内存使用量？

Answer 1

您既不会调用pthread_join()，也不会使用pthread_detach()，所以不会释放为该线程分配的资源。也就是说，每个线程都有自己的堆栈，这可能是导致内存消耗增加的原因。

关于实现的一些说明：由于您计划使用system()执行PHP脚本，并且实际上不需要处理共享变量或文件描述符，因此最好使用fork()和以下其中之一exec()的变体。这将产生一个新的进程，而无需创建线程的中间步骤。也建议不要使用system()，因为如果未正确清理输入内容，通常可以利用该程序。在这种情况下，如果您仅手动调用它可能会很好。