Valgrind: условный переход или перемещение зависит от неинициализированных значений: открытие файла - PullRequest
Новая
       14

Valgrind: условный переход или перемещение зависит от неинициализированных значений: открытие файла

1 голос
/ 30 марта 2019

[РЕДАКТИРОВАТЬ]: я добавил полный код.

Мне нужно создать простую версию команды "grep" в системах Unix на C. Все работает нормально, только Valgrind говорит Conditional jump or move depends on uninitialised value(s).
Я думаю, это может быть связано с файлом,что я пытаюсь открыть.Пожалуйста, смотрите мой код ниже.
Обратите внимание, что я не могу использовать <string.h> в своем коде.
Я компилирую код с помощью clang в Ubuntu:

cc -pedantic -Wall -Werror -g -std=c99 grep.c -o program

Вот что говорит Вальгринд: 

lukas@lukas-VirtualBox:~/Desktop/shared/Lab04/prg-hw04$ valgrind --track-origins=yes ./program Mem /proc/meminfo
==2588== Memcheck, a memory error detector
==2588== Copyright (C) 2002-2017, and GNU GPL'd, by Julian Seward et al.
==2588== Using Valgrind-3.13.0 and LibVEX; rerun with -h for copyright info
==2588== Command: ./program Mem /proc/meminfo
==2588== 
==2588== Conditional jump or move depends on uninitialised value(s)
==2588==    at 0x4C32D08: strlen (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==2588==    by 0x4EBC9D1: puts (ioputs.c:35)
==2588==    by 0x108970: check (grep.c:14)
==2588==    by 0x108AA9: read (grep.c:50)
==2588==    by 0x108B66: main (grep.c:71)
==2588==  Uninitialised value was created by a heap allocation
==2588==    at 0x4C2FB0F: malloc (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==2588==    by 0x108A04: read (grep.c:33)
==2588==    by 0x108B66: main (grep.c:71)
==2588== 
MemTotal:       10461696 kB
MemFree:         7701488 kB
MemAvailable:    8480772 kB
==2588== 
==2588== HEAP SUMMARY:
==2588==     in use at exit: 0 bytes in 0 blocks
==2588==   total heap usage: 4 allocs, 4 frees, 2,700 bytes allocated
==2588== 
==2588== All heap blocks were freed -- no leaks are possible
==2588== 
==2588== For counts of detected and suppressed errors, rerun with: -v
==2588== ERROR SUMMARY: 1 errors from 1 contexts (suppressed: 0 from 0)

Не могли бы вы помочь мне найти проблему?
Это мой файл grep.c. 

#include <stdio.h>
#include <stdlib.h>
#define SIZE 100
int printed = 1;    // return value -> 0 for patter found, 1 for pattern not found
char *pattern;
char *dest;

void check(char *line, int length, int size) {
    for (int i = 0; i < length; i++) {
        if (line[i] == pattern[0]) {
            for (int j = 1; j < size && (i+j) < length; j++) {
                if (line[i+j] == pattern[j]) {
                    if (j==size-1) {
                        printf("%s\n", line);   // print line
                        printed = 0;    // pattern found
                        goto END;
                    }
                } else {
                    break;
                }
            }
        }
    }
    END: ;
}

void read(void) {       // read lines, then check individual lines
    int c;
    int lengthPat = 0; 
    while(pattern[++lengthPat] != '\0');    // check length of pattern - I can't use string.h library
    FILE *file = fopen(dest, "r");
    size_t size =100;
    char *line = (char*)malloc(size * sizeof(char));
    if (line == NULL)   //succesfully created malloc?
        exit(102);
    int last = 0;
    if (file != NULL) {     // file succesfully opened
        while ((c = getc(file)) != EOF) {   
             if (c != '\n') {       // read line until \n
                if(last ==size)  {
                    char *p_line = realloc(line, 2*size*sizeof(char));
                    if (p_line == NULL)
                        free(line);
                    line = p_line;
                    size *= 2;
                }
                line[last++] = (char)c;
             }
             else {             // end of line, check for pattern
                check(line, last, lengthPat);
                last = 0; 
                for (int i = 0; i < size; i++) {    
                    line[i] = '\0';
                }
             }
        }
        fclose(file);
        free(line);
    }
    else {
    fprintf(stderr, "Error: Could not open file!\n");
    }
}

/* The main program */
int main(int argc, char *argv[])
{   
    if (argc == 3) {
        pattern = argv[1];
        dest = argv[2];
        read();
    }
    return printed;
}

Ответы [ 2 ]

1 голос
/ 30 марта 2019

В задаче отсутствовал нулевой терминатор \0 в конце строки line. Спасибо всем за помощь.

0 голосов
/ 30 марта 2019

«подключен к файлу» - это ключ.

Ваш входной файл содержит строки длиной более 100 символов.Замените массив стека динамически растущим массивом кучи. 

size_t size =100;
char c_line = malloc(size);

...
if(last ==size) 
     line = c_line = realloc(c_line, size<<=1);

При исправлении это ошибка в этой строке: 

printf("%s\n", line);   // print line

line не завершается нулем, поэтому использование printf являетсяпродвинутая темаМы делаем это вместо этого: 

for (int k = 0; k < length; k++)
    putc(line[k], stdout);
putc(line[k], stdout);
...