Dotaz: strncpy přidává '\031' – bug v glibc?

[stativ]

Právě jsem narazil na prapodivný problém, kdy mi v jednom konkrétním případě strncpy přidává znak '\031' (end of medium?) na konec řetězce.

Dělá mi to na desktopu s Archem (32 bit) – jak s gcc 4.4.2 tak s clang 2.6, tudíž předpokládám, že pokud bug není u mě tak bude nejspíš v glibc (2.11.1). Když jsem to ale zkoušel na mašině s OpenSuSE (64bit) tak to jelo správně.

Ukázka kódu, který blbne:

#include <stdio.h>
#include <string.h>
#include <stdlib.h>

int main() {
    char *test = "x^3 + (-x)^5";
    /*char *test = "sin(123 + 45 * ln(27 - 6))";*/
    char *str = malloc(strlen(test) * sizeof(char));

    strncpy(str, test, strlen(test));

    printf("%s\n",str);
}

Čekal bych, že na výstupu bude: x^3 + (-x)^5

ale já dostanu: x^3 + (-x)^5\031

Zvláštní je, že s jiným řetězcem (např. tím zakomentovaným to funguje).

Můj dotaz je – je bug u mě, nebo jinde?

---

---

Odpovědi

[0-58glfsdhesr]

"Prekladac automaticky pridava ke kazde konstatne typu *char znak \0, aby funkce poznaly, kde konci" (volne podle K&R). Avsak strlen vraci delku bez \0 (man 3 strlen).

V tomto pripade by mozna stalo za to pouzit funkci strdup, ale nekdy se hodi si pamet alokovat rucne.

Doporucuju na ten kod spustit valgrind, ktery odhali chyby podobneho typu.

[stativ]

Jop, přesně tak. Ani mě nenapadlo, že bych mohl takovou chybu udělat (zvlášť, když jinde to mám správně).

[stativ]

Safra, já zapomněl na plus jedna.

[KS]

Měl byste se raději podívat na význam základních operátorů v C++, protože sizeof(test) vrátí délku celého pole, namísto volání strlen(test), násobení sizeof(char) a přičítání jedničky. Nejen, že použití sizeof(test) je elegantnější, ale narozdíl od Vámi prezentovaného řešení má konstatní složitost (strlen(test) má lineární složitost!).

[AraxoN]

Neviem ako v C++, ale v C sizeof(test) vráti veľkosť dátového typu "smerník".

[Vojtěch Horký]

protože sizeof(test) vrátí délku celého pole

Jenže zde jde o pointer! Velikost celého pole může sizeof vrátit pouze pokud jde o pole v době deklarace. Prostě:

int a[10];
int *pa = a;
// sizeof(a) = 10*sizeof(int) != sizeof(pa) = sizeof(void *)

[stativ]

To je pěkný, ale je to pointer. Navíc sizeof vrací alokovanou paměť, ne tu, která je řetězcem reálně obsazená.