Question

编辑：所以你们更容易找到我的代码中我认为发生了段错误的地方，请查看名为/* Cache Structure */的评论部分

EDIT2：当gdb从gdb发出segfaults时，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;
    }

但不打印＆＃34;测试＆＃34;我在它下面。

ORIGINAL

因此在检查了segfault上的所有线程并使用gdb后，我无法弄清楚为什么我的程序在此行进行segfaulting。我正在写一个缓存模拟器（目前我的代码中只有直写）并且我没有包含我的方法，因为它们很好，它在主要方法中。

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

如果您想访问argc > 1，则需要测试argv[1]。程序始终会将其所调用的名称（例如您的./a.out）称为argv[0]，因此argc >= 1始终为true。要访问第一个真实参数 - 即argv的第二个元素 - 您需要测试是否至少有两个参数。但是，在访问> n时测试是否有argv[n]个元素更具可读性，因为两个数字都相同。

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

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之后 - 否则它将由太少的参数检查处理。）

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

在Cache Sim C程序中的SegFault

1 个答案: