Question

我有一个应用程序似乎正在执行我给它的所有代码，但仍然提示错误“运行时检查错误2 - 堆栈'val'已损坏。”我已经读过这可能是因为它正在从它的界限中得到一个赋值，但我检查了所有输入并且它们都是有效的。

我很好奇还有什么可能导致这种情况？

我觉得罪魁祸首可能在很多地方。 ExtractVals方法是我开始的地方，并且我添加了一个printf来测试索引，所有似乎都检查出来。没有出界的电话。此方法根据输入到方法的字符串（其为tolkenizer方法）将值重新分配给数组的每个元素。也许这可能导致错误？

任何提示都会很棒。

#include <stdio.h>
#include <stdlib.h>
#include <gl/glut.h>
#include <gl/gl.h>
#include <gl/glu.h>
#include <string.h>
#include <ctype.h>


void makeLower(char *input);
void extractVals(char *cmd, float *val);

FILE *file;
int g_mainWindow = -1;
float g_lightPos[] = {1, 1, -1, 0};
char commands [50][50];
int fileSize = -1;
int objectdrawn = 0;

int testint = 0;

void display()
{
    int i;
    char cmdTok[10] , *cmd = cmdTok;
    float val[5];
    char commandCpy[10];

    glClear(GL_COLOR_BUFFER_BIT | GL_DEPTH_BUFFER_BIT);
    glLoadIdentity();

    if ( objectdrawn == 0){

        for(i = 0 ; i <= fileSize ; i++){
            strcpy(commandCpy, commands[i]);
            printf("command = %s\n", commands[i]);
            cmd = strtok(commandCpy, " \n\0");
            printf("command = %s\n", commands[i]);
            switch(*cmd){
            case 'g'://translate object
                extractVals(cmd , val);
                glTranslatef(val[0] , val[1] , val[2]);
                break;
            case 's'://scales an object
                extractVals(cmd , val);
                if (val[4] == 1.){
                    glScalef(val[0],val[0],val[0]);
                }
                else if (val[4] == 3.){
                    glScalef(val[0] , val[1] , val [2]);
                }
                break;
            case 'r'://rotates an object
                break;
            case 'c'://this can call draw cone , cube, or change colors.
                if(strcmp(cmd , "cone") == 0){
                    //printf("drawing a cone\n");
                    glColor3f(1,0,0);
                    glutSolidCone(.5 , 1 , 8, 1);
                } else if (strcmp(cmd , "cube") == 0){
                    //glutSolidCube(1);
                } else if (*cmd == 'c'){
                    extractVals(cmd , val);
                    glColor3f(val[0] , val[1], val[2]);
                }
                break;
            case 't'://draw a torus or tea pot
                break;
            case 'o'://reads a meshfile
                break;
            case 'f'://save current frame buffer.
                break;
            case 'm':
                break;
            }

        }

        objectdrawn = 1;
        i = -1;
        printf("Loop Done!");
    }



    glFlush();
    glutSwapBuffers();
}

void reshape(int w, int h)
{
    float aspect = w / (float)h;

    glViewport(0,0,w,h);
    glMatrixMode(GL_PROJECTION_MATRIX);
    glLoadIdentity();
    glOrtho(-aspect, aspect, -1, 1, -1, 1);
    glMatrixMode(GL_MODELVIEW_MATRIX);
}

void idle()
{   
    /* parse a command from file */
    /* store the data for later draw */
    char linebyline [50], *lineStr = linebyline;
    int i=0;

    while(!feof(file) && file != NULL){
        fgets(lineStr , 50, file);
        makeLower(lineStr);
        strcpy(commands[i] , lineStr);
        printf("lineStr = %s\n", lineStr);
        printf("command = %s\n", commands[i]);
        fileSize = i;
        i++;

    }


    glutSetWindow(g_mainWindow);
    glutPostRedisplay();
}

void makeLower(char *input)
{
    while (*input != '\0')
    {
        *input = tolower(*input);
        input++;
    }
}

/*
Using a tolenizer this extracts out values needed for other functions to draw.
*/
void extractVals(char *cmd, float *val){
    int i=0;
    cmd = strtok(NULL, " ,");
    while(cmd != NULL){
        val[i] = atof(cmd);
        printf("val[%d] is %s\n", i , cmd);
        cmd = strtok(NULL, " ,");
        i++;
        testint++;
        printf("Ran this method %d times with a index of %d\n", testint, i);
    }

    //printf("Extracted values %f %f %f\n", val[0] , val[1] , val[2]);

    val[4] = i--;
}


int main(int argc, char **argv)
{
    file = fopen(argv[1], "r");

    glutInit(&argc, argv);

    glutInitDisplayMode(GLUT_RGBA | GLUT_DEPTH);
    g_mainWindow = glutCreateWindow("Hello, glut");
    glClearColor(0.5, 0.5, 0.5, 0);
    glEnable(GL_LIGHTING);
    glEnable(GL_LIGHT0);
    glEnable(GL_DEPTH_TEST);
    glLightfv(GL_LIGHT0, GL_POSITION, g_lightPos);

    glutDisplayFunc(display);
    glutReshapeFunc(reshape);
    glutIdleFunc(idle);

    glutMainLoop();
    fclose(file);
}

Vals的输出

val[0] is 0
val[1] is 0.5
val[2] is 0

val[0] is 0.25

val[0] is 1
val[1] is 1
val[2] is 1

val[0] is 4

val[0] is 0
val[1] is -0.5
val[2] is 0

val[0] is 1
val[1] is 1
val[2] is 4

val[0] is 1
val[1] is 1
val[2] is 0

Answer 1

这是Visual Studio帮助你:)）

你做了一些破坏堆栈的事情。

display（）中的

commandCpy []和cmdTok []是我开始寻找的第一个地方。

cmd []中未传递的字符串，传递给ExtractVals（），也是一个很好看的地方：）

逐步调试MSVC调试器中的代码 - 这应该可以直接解决问题！

Stack Around变量已损坏。这是什么意思？

1 个答案: