Question

当我运行此程序时，我收到以下错误：运行时检查失败＃2 - 变量'out_result'周围的堆栈已损坏。我试图打印out_result的长度来检查缓冲区是否有溢出，但长度始终小于CHAR_MAX_LENGTH = 512 有什么想法吗？

void GetCPUOutResult(FILE* cpu_out_file_pointer,char* out_result)
{
char temp[CHAR_MAX_LENGTH+1] = {0};
int char_index =fgetc(cpu_out_file_pointer) ,index=0;

while ( ((char)char_index) != '\n')
{
        temp[index]=(char)char_index;
        char_index =fgetc(cpu_out_file_pointer);
        index++;
}

temp[index]='\0';
sprintf_s(  out_result, 
            MAX_CMD_LINE_LENGTH, 
            temp, 
            NULL); 
}



void WorkerThreadFunction(THREAD_PARAMETERS* parameters)//LPVOID lpParam
{

//THREAD_PARAMETERS* parameters = (THREAD_PARAMETERS *) lpParam;
char *mission;
char cpu_file_name_in[CHAR_MAX_LENGTH];
char cpu_file_name_out[CHAR_MAX_LENGTH];
char out_result[CHAR_MAX_LENGTH+1]={0};
FILE *cpu_in_file_pointer, *cpu_out_file_pointer;
HANDLE mutexes_handles[NUM_MUTEXES];
int thread_id=parameters->thread_id;
char *log_file_name=parameters->log_file_name;
HANDLE youHaveWorkToDo=parameters->youHaveWorkToDoSemaphore[thread_id];
HANDLE readyToAcceptNewWork=parameters->readyToAcceptNewWorkSemaphore[thread_id];


//open file cpu<threadID>.in for writing

cpu_file_name_in[ 0 ] = '\0';
GenerateCPUFileName(cpu_file_name_in,"in",thread_id);
cpu_in_file_pointer = _fsopen(cpu_file_name_in, "w", _SH_DENYNO);


//open file cpu<id>.out for reading
cpu_file_name_out[ 0 ] = '\0';
GenerateCPUFileName(cpu_file_name_out,"out",thread_id);
cpu_out_file_pointer = _fsopen( cpu_file_name_out, "r", _SH_DENYNO );

while (TRUE)
{
    printf("Worker Thread: %d is waiting for work from dispatcher\n",thread_id);
    WaitForDispatcherToGiveMeWorkToDo(youHaveWorkToDo);
    THREADS_STATUS_ARRAY[thread_id]=0;/////////////////////////////////////////////////////////////////
    printf("Worker Thread: %d GOT work from dispatcher\n",thread_id);


    mission= WORKER_THREADS_BUFFER[thread_id];
    printf("Worker Thread: %d is executing mission %s\n",thread_id,mission);


    //pre mission execution writing to log file with lock

    mutexes_handles[0] = AquireSingleInstanceLock(FIRST_INSTANCE_MUTEX_NAME);//Set LOCK
    printf("Worker Thread: %d is trying to witre to INPUT LOGFILE\n",thread_id);
    WriteToLogFile(log_file_name,
                    thread_id,
                    " started running mission ",
                    mission);
    printf("Worker Thread: %d wrote successfully into INPUT LOGFILE\n",thread_id);
    ReleaseMutex( mutexes_handles[0] );//free LOCK


    //Write mission from worker thread buffer into cpu<id>.in
    sprintf(mission,"%s\n",mission);
    fwrite( mission, sizeof( char ), strlen(mission), cpu_in_file_pointer);
    fflush( cpu_in_file_pointer );//flush data from accoiated buffer to the file
    printf("Worker Thread: %d wrote successfully in cpu%d.in core\n",thread_id,thread_id);


    //get result from cpu<id>.out
    printf("Worker Thread: %d is trying to get result from CPU<Id>.OUT\n",thread_id);


    Sleep(50);
    GetCPUOutResult(cpu_out_file_pointer,out_result);
    printf("Worker Thread: %d got result:%s from CPU<Id>.OUT\n result length is:%d\n",thread_id,out_result,strlen(out_result));

    ///post mission execution writing to log file with lock
    printf("Worker Thread: %d trying to write result:%s into OUT LOG FILE\n",thread_id,out_result);
    mutexes_handles[1] = AquireSingleInstanceLock(SECOND_INSTANCE_MUTEX_NAME);//Set LOCK
    WriteToLogFile(log_file_name,thread_id," is done! Mission result is ",out_result);
    ReleaseMutex( mutexes_handles[0] );//free LOCK
    printf("Worker Thread: %d successfully wrote result:%s into OUT LOG FILE\n",thread_id,out_result);

    printf("Worker Thread: %d successfully executed mission:%s \n",thread_id,mission);

    if (strcmp(mission,"quit\n") == 0  )
    {
        break;  
    }
    else
    {
        InformDispatcherThreadIsReadyToAcceptWork(readyToAcceptNewWork,
                                              (LPLONG)&THREADS_STATUS_ARRAY[thread_id]);
        THREADS_STATUS_ARRAY[thread_id]=1;/////////////////////////////////////////////////////////////////
    }
}
return;

}

Answer 1

更改此行

sprintf_s(out_result, MAX_CMD_LINE_LENGTH, 
        temp, 
        NULL);

成为这个：

sprintf_s(out_result, CHAR_MAX_LENGTH, 
        "%s"
        temp);

有关sprintf_s()的详细信息，请在此处阅读：http://msdn.microsoft.com/en-us/library/ce3zzk1k.aspx（页面末尾也有示例）

运行时检查失败＃2 - 变量堆栈---已损坏

1 个答案: