c - Cache Sim C 程序中的 SegFault

Question

编辑:这样你们就可以更容易地找到我认为发生段错误的代码中的位置，请查看名为 /* Cache Structure */

的注释部分

EDIT2:使用 gdb 犯了一个巨大的错误，当 gdb 出现段错误时，正确的输出是:

(gdb) run
Starting program: /.autofs/ilab/ilab_users/petejodo/Assignment4/a.out 
1
/.autofs/ilab/ilab_users/petejodo/Assignment4/a.out

Program received signal SIGSEGV, Segmentation fault.
0x00449453 in strlen () from /lib/libc.so.6
(gdb) 

EDIT3:在采用 n00b 策略(例如打印行)之后，我发现它之前打印过:

file = fopen(purefile, "r");
    if (file == 0){
        printf ("Could not find file!\n");
        return 0;
    }

但不打印我在其下面的“测试”。

原版

因此，在检查了段错误上的所有线程并使用 gdb 之后，我无法弄清楚为什么我的程序在这一行出现段错误。我正在编写一个缓存模拟器(目前我的代码中只有直写功能)，并且我没有包含我的方法，因为它们很好，它位于主方法中。

int main(int argc, char **argv) {
    FILE* file;

    /* Counter variables */
    int i;
    int j;

    /* Helper Variables */
    int setAdd;
    int totalSet;
    int trash;
    int size;
    int extra;
    char rw;

    /* Necessary Character Arrays */
    char hex[100];
    char bin[100];
    char origTag[100];
    char bbits[100];
    char sbits[100];
    char tbits[100];

    /* Cache Info Variables */
    int setNumber = 4096; /* cacheSize/blockSize : (16,384/4) */
    int setBits = 12; /* log(setNumber)/log(2) : (log(4096)/log(2)) */
    int tagSize = 18; /* 32-(blockBits + setBits) **blockBits = log(blockSize)/log(2)** : (32 - (2 + 12) */

    /* Results */
    int cacheHit = 0;
    int cacheMiss = 0;
    int write = 0;
    int read = 0;

    /* Cache Structure */
    tempLine cache[4096];

    char* style;
    char* purefile;
    if (strcmp(argv[1], "-h")==0) 
    {
        puts("Usage: sim <write policy> <trace file>");
        return 0;
    } 
    style = argv[1];
    purefile = argv[2];
    file = fopen(purefile, "r");/* HYPOTHESIZED SEGFAULT HERE */
    if (file == 0){
        printf ("Could not find file!\n");
        return 0;
    }
    printf("test1");
    /* Setting Structure Default Values */
    for(i = 0; i < setNumber; i++)
    {
        cache[i].tag = (char *)malloc(sizeof(char)*(tagSize + 1));
        for(j = 0; j < tagSize; j++)
        {
            cache[i].tag[j] = '0';
        }
        cache[i].valid = 0;
    }

    /* Main Loop */
    while(fgetc(file) != '#')
    {
        setAdd = 0;
        totalSet = 0;

        fseek(file, -1, SEEK_CUR);
        fscanf(file, "%d: %c %s\n", &trash, &rw, origTag);

        /* Cutting off '0x' off from address '0x00000000' and adding 0's if necessary */
        size = strlen(origTag);
        extra = (10 - size);
        for(i = 0; i < extra; i++)
            hex[i] = '0';
        for(i = extra, j = 0; i < (size-(2-extra)); i++, j++)
            hex[i] = origTag[j + 2];

        hex[8] = '\0';

        hex2bin(hex, bin);

        split(bin, bbits, sbits, tbits);

        /* Changing cArray into int */
        for(i = 0, j = (setBits - 1); i < setBits; i++, j--)
        {
            if (sbits[i] == '1')
                setAdd = 1;
            if (sbits[i] == '0')
                setAdd = 0;
            setAdd = setAdd * pow(2, j);
            totalSet += setAdd;
        }

        /* Calculating Hits and Misses */
        if (cache[totalSet].valid == 0)
        {
            cache[totalSet].valid = 1;
            strcpy(cache[totalSet].tag, tbits);
        }

        if ((cache[totalSet].valid == 1) && (strcmp(cache[totalSet].tag, tbits) == 0))
        {
            /* HIT */
            if (rw == 'W')
            {
                cacheHit++;        
                write++;
            }
            if (rw == 'R')
                cacheHit++;    
         }
         else
         {
             /* MISS */
            if (rw == 'R')
            {
                cacheMiss++;
                read++;
            }
            if (rw == 'W')
            {
                cacheMiss++;
                read++;
                write++;
            }
            cache[totalSet].valid = 1;
            strcpy(cache[totalSet].tag, tbits);
        }
        /* End Calculations */  
    }
    printResult(cacheHit, cacheMiss, read, write);

    return 0;
}

我从 gdb 得到的是这样的:

**INCORRECT**

(gdb) run
Starting program: /.autofs/ilab/ilab_users/petejodo/Assignment4/a.out 

Program received signal SIGSEGV, Segmentation fault.
0x080489b6 in main (argc=1, argv=0xbfffe954) at sim.c:128
128             if (strcmp(argv[1], "-h")==0)
(gdb) bt

#0  0x080489b6 in main (argc=1, argv=0xbfffe954) at sim.c:128

我有点迷失了，任何帮助都会很棒。谢谢!

哦，还要检查 argv[1] 的值，它是 NULL 或 0x0 因为我认为就是因为这个。 argv[1] 应该包含帮助标志或直写或回写，我只是还没有编码是否检查直写或回写。

Answer 1

如果要访问argv[1]，您需要测试argc > 1。程序始终接收其调用的名称(例如，在您的情况下为 ./a.out)作为 argv[0]，因此 argc >= 1 始终为真。要访问第一个实参 - 即 argv 的第二个元素 - 您需要测试是否至少有两个参数。但是，在访问 argv[n] 时测试是否有 > n 元素在 IMO 中更具可读性，因为两个数字相同。

以下是有关如何更改代码的示例:

if (argc < 3 || (argv > 1 && strcmp(argv[1], "-h")==0))
{
    puts("Usage: sim <write policy> <trace file>");
    return 0;
} 

在这种情况下，如果没有提供足够的参数来包含有效的 argv[1] 和 argv[2] 元素，或者如果第一个参数是 -h (以防有人在 -h 之后添加内容 - 否则它将由 Too-few-arguments 检查来处理)。

您应该看看getopt()顺便说一句 - 它使参数/开关解析变得更加容易和清晰。