C言語：引数で渡した文字列リテラルに文字をコピーしたい

Question

以下のようなコードを作成したのですが、引数に渡したバッファに文字列をコピーすることができません。。。
NULLが返ってきてしまうのですが、このように引数にアドレスを渡してそこに値を書き込んで返してもらうような処理はどのように修正すればよいでしょうか。

ご教示くださいますと幸いです。よろしくお願いいたします。

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

int copy(char *buf)
{
        char *src = "TEST\n";
        buf = (char *)malloc(24);
        memset(buf, 0, 20);
        sprintf(buf, "%s", src);
        return 0;
}

int main(void)
{
        char *dst;
        int ret;

        ret = copy(dst);
        printf("%s", dst);
        free(dst);

        return 0;
}

Answer 1

僕ならこうする:

#define _CRT_SECURE_NO_WARNINGS /* disable warning for VC++ */
#include <stdio.h>
#include <stdlib.h>
#include <string.h>

char* copy(const char* src) {
    char* buf = (char*)malloc(strlen(src)+1);
    if ( buf != NULL ) {
      strcpy(buf, src);
    }
    return buf;
}

int main(void) {
    char* dst = copy("TEST");
    printf("%s", dst);
    free(dst);

    return 0;
}

Comments

[Ykkykk]

ご回答ありがとうございます。
確かにコピーするだけなら頂いたご回答の方が圧倒的にスマートですね。。。
大変勉強になりました。ありがとうございます！

[episteme]

で、これとおんなじもんが標準関数にあるんよ。

https://linuxjm.osdn.jp/html/LDP_man-pages/man3/strdup.3.html

[Ykkykk]

そうだったのですね。わざわざお教えくださりありがとうございます。。。
大変勉強になります！

Answer 2

呼び出し先で領域を確保するってのは筋が悪いです
やるなら、呼び出し元で確保し、あとの処理もするようにしたほうがいいです

確保した領域のアドレスと、確保したサイズを関数に渡して処理するようにしましょう

Comments

[Ykkykk]

ご確認いただきありがとうございます。
呼び出し元で領域を確保した方が良いのですね。。。
基本的なことをお教えいただきありがとうございます！勉強になりました！

[ponpoko1968]

まずは直近でわからないことに答えてあげる方が良いのかと思いました。y_waiwaiさんがおっしゃることは全くもって正しいのですが、とりあえず現状で意図したものが少なくとも意図したようにうごくようにならないと、なぜこの作りだと一般的に使える関数にならないかという課題に行き着くという、次のステップには進めないと思うんですよ。

Answer 3

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

int copy(char **buf)
{
  char *src = "TEST\n";
  size_t len = strlen(src);
  *buf = (char *)malloc(len);
  memset(*buf, 0, len);
  sprintf(*buf, "%s", src);
  return 0;
}

int main(void)
{
  char *dst;
  int ret;

  ret = copy(&dst);
  printf("%s", dst);
  free(dst);

  return 0;
}

Comments

[Ykkykk]

ご回答ありがとうございます。ポインタのポインタにするのですね。。
理解できました。ありがとうございます。

[episteme]

> *buf = (char *)malloc(strlen(src));
1byte足りない

> memset(*buf, 0, 20);
オーバーランしてる

[ponpoko1968]

すいません（^^ゞ

[ponpoko1968]

超有名人に指摘されて恐縮です。。

[episteme]

直後にsprintfしてんだからmemset要らなくね?

[ponpoko1968]

見落としてました。要らないですね。もうちょっとちゃんと見落として添削すべきでした。