文字列の一部を別の文字列にコピーする

Question

文字列の一部を別の文字列にコピーする際に問題があります。これら二つのcharのポインタを考える：私はstrncpyを中にセグメンテーションフォールトを取得

char* findUsername(char* line, char* username){ 
    char* ptr = strstr(line, ":"); 
    ptrdiff_t index = ptr - line; 
    strncpy(username, line, index); 

    return username; 
} 

：

line points at string cointaining: "helmutDownforce:1234:44:yes" 
username points at: NULL 

は、ここで入力としてこれらのポインタを取る私の機能です。どうして？私が望む結果は、helmutDownforceを含む文字列へのポインタを返す関数です。

Answer 1

strncpyのmanualによると：

the destination string dest must be large enough to receive the copy 

ですから、strncpyを呼び出す前にusernameためmallocで最初のいくつかのメモリを割り当てる必要があります。

Answer 2

この関数は新しい文字列を割り当てて返します。したがって、メモリリークを避けるために、呼び出し関数は最終的に解放する必要があります。行にセパレータコロンがない場合は、NULLを返します。

char* findUsername(char* line){ 
    char* ptr = strchr(line, ':'); 
    /* check to make sure colon is there */ 
    if (ptr == NULL) { 
     return NULL; 
    } 

    int length = ptr - line; 
    char *username = malloc(length+1); 

    /* make sure allocation succeeded */ 
    if (username == NULL) return NULL; 

    memcpy(username, line, length); 
    username[length] = '\0'; 
    return username; 
}