C - неадресуемый байт для строки динамической памяти - PullRequest
0 голосов
/ 14 декабря 2018

У меня есть такой контекст:

char* bname(char const *mypath) {    //basename
    char* bnm = (char*) malloc((strlen(mypath)+1)*sizeof(char));
    char lim='/';
    char* z = strrchr(mypath, lim);
    if (z) {
        strcpy(bnm, z + 1);
    } else {
        strcpy(bnm,mypath);
    }
    return bnm;
}

void doX(int sockfd, char* filename) {
    if (send(sockfd, filename, 1024, 0) == -1) {
        // exit;
    }

    /*
    No error with this one:
    if (send(sockfd, "test", 1024, 0) == -1) {
        // exit
    }
    */
}

Который вызывается исключительно в основном так:

// argv[3] = test.jpg
char* fname= bname(argv[3]);
doX(socketd, fname);
free(fname);

Компиляция: gcc -Wall -Werror -pedantic -std = c99 -ggdb3

Valgrind: valgrind --leak-check = full --tool = memcheck

Syscall param socketcall.sendto(msg) points to unaddressable byte(s)
==7096==    at 0x4F5EC4D: send (send.c:28)
==7096==    by 0x109197: doX (client.c:94)
==7096==    by 0x1093A0: main (client.c:146)
==7096==  Address 0x522e2f9 is 0 bytes after a block of size 9 alloc'd
==7096==    at 0x4C2FB0F: malloc (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==7096==    by 0x108F38: bname (client.c:43)
==7096==    by 0x109378: main (client.c:145)

Не могу найти причину этого предупреждения, скорее всего, это отправка () из doX, поскольку, если я дам ей буквальную строку, предупреждение не появится.

Ваша помощь приветствуется.

1 Ответ

0 голосов
/ 14 декабря 2018

измените его следующим образом:

if (send(sockfd, filename, strlen(filename)+1, 0) == -1) {

имени файла может не хватить памяти (1024 байта)

...