c言語 動的メモリの確保

Question

###前提・実現したいこと
c言語についての質問です。
実行時にリダイレクトでファイルの内容を出力するプログラムを書きました。
動的メモリの確保を使ってみたのですが
5 abcde
20 how-is-your-progress
Aborted (コアダンプ)
のようにエラーが出て正常に動きません。どこが間違っているのでしょうか？

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

5 abcde
20 how-is-your-progress
Aborted (コアダンプ)

###該当のソースコード

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

int main(void)
{
	int n;
	char *str;
	
	while(scanf("%d %s",&n,str=(char*)malloc(sizeof(char)*(n+1)) )!=EOF){
		printf("%d %s\n",n,str);
		free(str);
	}
	
	return 0;
}

Answer 1

こんにちは。

scanf()から戻ってきた時にはnに値が入ってますが、scanf()呼び出しの時点ではnに値は入っていません。
そして、str=(char*)malloc(sizeof(char)*(n+1))はscanf()を呼び出す時に実行されるため、nが未定のまま呼ばれます。なので何が起こるか分かりません。

あと、scanf()の戻り値を再確認した方が良いですよ。EOFが返って来なかったら、正常に読み出されているという判定は誤りです。

更に、EOFが返ってくるとメモリ・リークするかも。（このままでは動かないので修正次第ですが。）
リークしないように作るために、スマートな書き方ができないことも少なくないです。

Answer 2

nに値が設定されないうちに参照されています。
また、scanfで文字列長が指定されていません。