なぜこのstrstrのインスタンスは私にセグメンテーションフォールトを与えていますか？

Question

ここ

は完全なコード

int count_substr(const char *str, const char *sub) 
{ 
    char *ret; 

    int a = 0; // used to tell where the pointer has moved to 
    int count = 0; 

    ret = strstr(str, sub); 

    while (strstr(ret, sub) != NULL) { 
     printf("The substring is: %s\n", ret); 

     for (a = 0; a < strlen(sub); a++) { 
      ret++; 
     } 

     printf("The substring after moving pointer is: %s\n", ret); 
     count++; 
    } 

    return count - 1; 
} 

私は、私がかつて

strstr(ret,sub) 

がゼロになるNULLポインタを使用していないよ、ここで何が起こっているか理解していないので、なぜそれは私にワンセグ障害を与えているです？

Valgrindのは

サイズの不正な読み取り1と アドレス0x0は（最近）、stack'd mallocで確保するかしないと述べているでしょうが、最初の呼び出しret = strstr(str,sub);場合はテストしていない

Answer 1

をfree'd成功する。

NULLが返された場合、次のコールstrstr(ret,sub)は確実に未定義の動作を呼び出します。

さらに、あなたのコードがwhileループで正しくretを更新しません、あなたは、単に試合の長さによって、それを進めない、試合過去の最初の文字にretを設定する必要があります。ここでは簡単なバージョンは次のとおりです。

int count_substr(const char *str, const char *sub) { 
    /* returns the number of non overlapping matches of sub in str */ 
    const char *ret = str; 
    int count = 0; 

    if (ret && sub && *sub != '\0') { 
     while ((ret = strstr(ret, sub)) != NULL) { 
      ret += strlen(sub); 
      count++; 
     } 
    } 
    return count; 
} 

Answer 2

あなたは間違いなく（（3）のマニュアルはstrstrを）はstrstrの戻り値をチェックする必要があります

RETURN VALUE 
     These functions return a pointer to the beginning 
     of the located substring, or NULL if the substring is not found.