【C言語】ディープコピーの方法について

Question

実現したいこと

以下のソースでAにBをディープコピー？
(ポインタの先の内容をコピー)したいのですが、
うまくいっていません。
bstringCopy()内で実装できるディープコピーの手法を
ご教授いただけませんでしょうか。

発生している問題・エラーメッセージ

Segmentation fault(コアダンプ)

該当のソースコード

test.h
typedef struct tagbstring* bstring;

struct tagbstring {
    int mlen;
    int slen;
    unsigned char* data;
}


test.c

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

void bstringCopy(bstring dst, bstring src){
    dst->mlen = src->mlen;
    dst->slen = src->slen;

    size_t byte = sizeof(char);
    size_t size = byte * strlen(src->data) + byte;
    dst->data = malloc(size);
    strcpy(dst->data, src->data);

    return;
}

int main(){
    bstring a;
    bstring b;

    bstringCopy(a, b);

    return 0;
}

Comments

[y_waiwai]

提示のコードでどういう不具合があるんでしょうか。
そして、それをどう言う風にしたいんでしょう。

Answer 1

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

typedef struct tagbstring* bstring;

struct tagbstring {
    int mlen;
    int slen;
    unsigned char* data;
};

bstring bstringMake(int m, int s, const char* d) {
  bstring tmp = malloc(sizeof(struct tagbstring));
  if ( tmp != NULL ) {
    char* str = malloc(strlen(d)+1);
    if ( str == NULL ) {
      free(tmp);
      tmp = NULL;
    } else {
      strcpy(str, d);
      tmp->data = str;
      tmp->mlen = m;
      tmp->slen = s;
    }
  }
  return tmp;
}

void bstringDestroy(bstring b) {
  free(b->data);
  free(b);
}

void bstringCopy(bstring dst, bstring src){
    dst->mlen = src->mlen;
    dst->slen = src->slen;

    size_t byte = sizeof(char);
    size_t size = byte * strlen(src->data) + byte;
    dst->data = malloc(size);
    free(dst->data);
    strcpy(dst->data, src->data);

    return;
}

int main(){
    bstring a = bstringMake(123, 456, "Hello, World");
    bstring b = bstringMake(0, 0, "");

    bstringCopy(b, a);

    printf("a: data = [%s] mlen=%d slen=%d\n", a->data, a->mlen, a->slen);
    printf("b: data = [%s] mlen=%d slen=%d\n", b->data, b->mlen, b->slen);

    bstringDestroy(a);
    bstringDestroy(b);
    return 0;
}

Answer 2

deep copy以前の段階で間違っています。

typedef struct tagbstring* bstring;
としているのですから、bstring型の変数は構造体へのポインタです。ですから
bstring a;
bstring b;
のaやbもポインタです。そのポインタを初期化(何かを明示的に指すようにすること)もなくポインタが指す先にアクセスする行為は未定義ですが、多くの場合アクセス違反(Segmentation fault)を引き起こすもので、実際そうなったという話です。
当然ながら、tagbstringのメンバー変数のポインタ(data)も適切な初期化が行われた後でなければそのポイント先にアクセスしてはいけません。

deep copyの処理自体は間違っているとまでは言いませんが、
・sizeof(char)はsizeofの動作の定義から必ず1です。

JISX3010
6.5.3.4 sizeof演算子  
(略)
型char，unsigned char若しくはsigned char（又はそれらの修飾版）をもつオペランドに適用した場合の結果は，1とする。

・mallocに失敗する可能性は0ではない、とするのがお作法です。そして、mallocに失敗したら適宜処理(といっても強制終了するぐらいしか手はないことが多いでしょうが)すぺきです。

というあたりに検討の余地があると思います。

Comments

[thkana]

うげ、なんかアイコンがあひるちゃんに入れ替わってる...(治るんだろうなぁ)

おっと、コメントに気がついてなかった。
> ローカル変数a,bには値が入っているものだという認識でお願いいたします。
ちゃんと値が入っていればsegmentation faultにはならないはずなので、その認識は無理です。
それと、全体に影響のある追記は質問本文にも行ってください。

[thkana]

(アイコンの入れ替わりはブラウザ変えたらでない...firefoxダメ?)

[y_waiwai]

ちょっと前から、アイコンが入れ替わったり、ベストアンサーの赤色が別のところにつく、って不具合が発生してますねー
困ったもんです

[ikadzuchi]

最近見かける気がするこの不具合、前からあったのかと気になっていましたが、やっぱり最近でいいんですね。
1年経ってまだ着々と破壊が進んでいるんですねえ。

Answer 3

そもそも、ローカル変数で定義したものは中身はデタラメです
それをコピーしようとすることが間違いです

Comments

[decman]

回答ありがとうございます。
すみません、記載が漏れておりましたが、ローカル変数a,bには
値が入っているものだという認識でお願いいたします。

[y_waiwai]

提示のコードではどういう不具合があるんでしょうか

[decman]

不具合といいますか、そもそもディープコピーの手法がイマイチ理解できていないので、
この書き方で合っているかもわかっておりません。。
なので、この構造体の場合におけるディープコピーの方法を
お教えいただければと思っております。。

[y_waiwai]

なら、それぞれの変数での値の代入からの実際に動作するコードを書いてみてください。
そのコードさえないので、答えようがないです

[jimbe]

『ディープコピーの手法』なんて、ただ必要なコピーをするというだけですけど…なにか試験問題のように、ディープコピーの正解があると考えているのでしょうか。