C言語でSegmentation fault発生

Question

8byteの文字列の初期化を試みています。
各バイトをFF(16進)で埋めるためにunsigned charにキャストしています。
（charでは、-128 ～ 127しか表現できないため）
その後、memsetで全バイトを0xffで埋めているのですがうまくいきません
何が原因でどのような対処方法がありございますでしょうか

リンク内容

void main(void)
{
    char str[] ="abcdefte";
    set_str(str);
    return;
}

void set_str(char *str)
{
    unsigned char uchr;
    uchr = (unsigned char)str;
    memset( uchr, 0xff, sizeof(str));
}

Comments

[cateye]

"abcdefte"は、文字列リテラルといって変更不可です。
参考：“文字列リテラルとはなんですか？”https://teratail.com/questions/20564

[maisumakun]

> "abcdefte"は、文字列リテラルといって変更不可です。

ではないです。ここではchar[]の初期化子なので、単なる配列として書き換え可能です。

Answer 1

いろいろおかしいです。

#include <string.h>

void set_str(char *str, size_t size)
{
    unsigned char *uchr;
    uchr = (unsigned char *)str;
    memset( uchr, 0xff, size);

    // キャストせずに下記でも問題なし。というか、普通はこう。
    // memset( str, 0xff, size);

}

int main(void)
{
    char str[] ="abcdefte";
    set_str(str, sizeof str);
    return 0;
}

・memsetの第一引数はポインターなのにポインターを渡してない（セグメントフォールトの直接原因）
・関数に引数で渡された配列のサイズは関数側では不明なので、呼び出し側から渡す必要あり　(sizeof(str)はポインターのサイズを表す）
・mainの返り値はint

Comments

[pokemonta]

私のPGM何がおかしいかったでしょうか？？
関数呼び出し側でsizeof strが必要なんでしょうか？
私は定義側でsizeof 使っています

[otn]

おかしい点を補足しました。

[pokemonta]

＞関数に引数で渡された配列のサイズは関数側では不明なので、呼び出し側から渡す必要あり
あ、そうなんですね。
文字列（abcdefte）をそのまま渡しているので
関数側は、それをカウントすればいけるのかな～と思ってました。
pythoんはいけるはず

[otn]

> 文字列（abcdefte）をそのまま渡しているので
いいえ。
渡しているのは、「文字列の先頭アドレス」だけです。

> pythoんはいけるはず
C以外の多くの言語では、文字列自体が長さの属性を持っています。

[pokemonta]

>渡しているのは、「文字列の先頭アドレス」だけです。
ええ、知らなかったっす。。。ええ

Answer 2

"abcdefte"は、文字列リテラルと言って変更不可です。
参考：“文字列リテラルとはなんですか？”https://teratail.com/questions/20564
また、sizeof(str)では、ポインタのサイズ（64ビット環境なら８）しか求まりません。

Comments

[otn]

char *str ="abcdefte";
じゃなくて、
char str[] ="abcdefte";
なので、書き換えているのはchar配列です。

[fana]

本件はリテラルを書き換えているわけではないです．

[cateye]

失礼、間違いました。m(_"_)m

Answer 3

uchr = (unsigned char)str;

これは何をしたいのでしょう？

strは char*型であり，おそらくは32bitとか64bitとかの値です．
unsigned charが8bitだとしたら，これをunsigned charにしろと言われても当然入りきらないですから，下位8bit分しか残らない結果になるでしょう．

そうして著しく小さい値に切り詰められてしまったであろう値を memset にアドレスとして与えているとしたら，まぁ，Segmentation fault になり得るでしょう．

---

あと，それとは別に

memset( uchr, 0xff, sizeof(str));

これの最後の引数も記述として間違っているでしょう．（たまたま 「8byte」 という値として一致することになり得るかもしれないけれども）

Answer 4

void set_str(char *str, size_t n)
{
    unsigned char* uchr;
    uchr = (unsigned char*)str;
    memset( uchr, 0xff, n);
}

uchrをポインタにしました。これがエラーと関係あります。
引数に文字列の長さを追加、これは今回のエラーとは関係ありません。ただ無いと別の不具合が出ます。

そして、そもそもですが

各バイトをFF(16進)で埋めるためにunsigned charにキャストしています。

キャストしなくてもmemsetできます。