文字列の中から指定の文字列を抽出する関数でなにも表示されない

Question

ある文字列Aの中にある文字列Bを抽出するというのをやっているのですが、何も表示されんません。
どのように修正するべきでしょうか？
A＝tokyo_university
B=university

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

char select_univ(char name[]);

int main(){
  char name[17]="tokyo_university";
  char extraction[10];
  char *p;
  p=select_univ(name);

  if(p==NULL){
    printf("Not found\n");
    return(1);
  }

  strcpy(extraction,p);
  printf("%s\n",extraction);

  return(0);
}

char select_univ(char name[]){
  char *a;
  a=strstr(name,"university");
  return(name);
}

エラーは発生しておらず、ワーニングが出ています。
integerなのでintに変える必要があるのかと思いpとaをintに変えましたが解決しませんでした。

ext.c: In function 'main':
ext.c:13:4: warning: assignment makes pointer from integer without a cast [-Wint-conversion]
   p=select_univ(name);
    ^
ext.c: In function 'select_univ':
ext.c:29:9: warning: return makes integer from pointer without a cast [-Wint-conversion]
   return(name);

Answer 1

関数select_univの戻り値の型はcharつまり整数として宣言されていますね。
これは文字列を指すポインタでなければならないのではないですか？

そして、せっかくaに値をセットしているのにreturn nameしちゃってますね。
これもおかしい。

あと、ついでにもう一つ指摘すると、

strcpy(extraction,p);

ですが、これはpがextractionのサイズより長い文字列だと重大なアクセス違反が起こります。

Comments

[xeno]

確かにそうでした
戻り値にしろ、ポインタにしろ勘違いしていました。
ありがとうございます。

Answer 2

char select_univ(char name[]);

この関数の戻り値はchar と定義されてます

char *a;

a=strstr(name,"university");

が、char* の変数に代入されてます。これ、おかしいんじゃね？とコンパイラさんがおっしゃってます。

この関数で、「なにを」戻したいんでしょうか。そこらへんはっきりさせましょう