C 言語strcpyについて

Question

線形リストを作ってその構造体の中にpathname配列に格納されている文字列を記憶させるプログラムを作っているのですが警告が出てきてうまく動作してくれません。"stack is success、"stack is not successの出力がされないのでそのあたりが間違っていると思います。全体ではないのですが該当箇所のプログラムを載せます。気になる部分はstrcpyの書式についてです。先頭アドレスを引数として渡すようなのですがこの書き方ではまずいですか？エラーを見ると配列pathnameの宣言が間違っていてポインタを用いるのかな？と思うのですがよくわかりません。わかる方がいたら教えていただけると幸いです。よろしくお願いします。

コード
typedef struct nodetag{
  char data[512];
  struct nodetag *next;
} node_t;
int stack(char,node_t *);
char pathname[512];
node_t *p;

  printf("Command is pushd\n");
        memset(pathname,'\0',512);
        getcwd(pathname,512);
        if (stack(pathname,p) == 0) {
          printf("stack is success\n");
        }else{
          printf("stack is not success\n");
        }


int stack(char pathname,node_t *p){
    node_t *nd;
    nd = malloc(sizeof (node_t));
    if (nd == NULL) {
      return 1;
    }else{
      nd -> next = p;
      strcpy(nd -> data,pathname);
      p = nd;
      //リストの先頭に追加
      printf("nd -> data is %s\n",nd -> data );
      return 0;
    }

線形リスト
コンパイル時に出てくるエラー
simple_shell.c:326:19: warning: incompatible pointer to integer conversion
passing 'char [512]' to parameter of type 'char' [-Wint-conversion]
if (stack(pathname,p) == 0) {
^~~~~~~~
simple_shell.c:31:15: note: passing argument to parameter here
int stack(char,node_t *);
^
simple_shell.c:396:25: warning: incompatible integer to pointer conversion
passing 'char' to parameter of type 'const char *'; take the address with
& [-Wint-conversion]
strcpy(nd -> data,pathname);
^~~~~~~~
&
/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.12.sdk/usr/include/secure/_string.h:83:33: note:
expanded from macro 'strcpy'
__builtin___strcpy_chk (dest, src, __darwin_obsz (dest))
^~~

Comments

[退会済みユーザー]

質問の編集画面の下の方に「Markdown記法とは」って言うのがあると思うから、「コードの入力」って言うのを参考にコード部分を書き換えてね。

[tekondo]

ご指摘ありがとうございます。こんな機能があるんですね

Answer 1

コンパイルと実行はできる様にしました

nd -> next = p;
strcpy(nd -> data,pathname);
p = nd;

この辺は今後たぶん直されるのかなと思います。

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

typedef struct nodetag{
    char data[512];
    struct nodetag *next;
} node_t;

int stack(char *,node_t *);

void main (void){
    char pathname[512];
    node_t *p;

    printf("Command is pushd\n");
    memset(pathname,'\0',512);
    getcwd(pathname,512);
    if (stack(pathname,p) == 0) {
        printf("stack is success\n");
    }else{
        printf("stack is not success\n");
    }
}

int stack(char *pathname,node_t *p){
    node_t *nd;
    nd = malloc(sizeof(node_t));
    if (nd == NULL) {
        return 1;
    }else{
        nd -> next = p;
        strcpy(nd -> data,pathname);
        p = nd;
        //リストの先頭に追加
        printf("nd -> data is %s\n",nd -> data );
    }
    return 0;
}

Comments

[tekondo]

ありがとうございます！変更後の書式のようにpathnameを後から＊をつけてポインタとして扱うこともできるのですね！使うのはいいもののポインタについてよく理解できていないのが今回の問題を通してわかったので勉強していきます。ご回答ありがとうございました。

[A.Ichi]

最初、node_t *p;ではポインターを定義しただけなので、node_t *p =NULL;とされたほうがNULL値となり確実です。
勉強頑張ってください。

Answer 2

stack()関数の第１パラメータ、pathnameがcharとなっていますが、これだとchar型の１バイトのデータとなります。まず、それは期待している動作（というか、仕様）ですか？pathnameが512バイトの文字列データの変数なので、文字列として渡したいのではないでしょうか？
コンパイル時のwarnningもこれに起因しています。strcpyのパラメータはどちらもchar *とchar型のポインタですが、これはコピー先のアドレスとコピー元のアドレスになります。warnningが出ているのは第２パラメータ（＝コピー元）の方で、アドレスを渡さなければならないところ、char型のデータを渡してしまっているからです。

ここまで書けば、どうすべきかはわかると思います。

Comments

[tekondo]

pathnameの記述で先頭アドレスをさせていると思ってしまいました、、、勉強不足ですありがとうございました！