書式文字列、char型の変換について

Question

###前提・実現したいこと
お世話になります。intだけでなく型を意識するということで、char型配列に数字を格納して、その後演算できる数値として出力するプログラムを考えました。
4つの数字を一度に入れて4つの大小を比較するプログラムにする予定です。

配列の要素にそれぞれ数字を格納したいため、int型配列は使えないですよね？
先日の配列で使用したgetchなどはまだいまいち理解できなく一から作成するには使えずにいます。

###発生している問題・エラーメッセージ
以下のコードを読んだ際に警告が出てしまいます。

Run-Time Check Failure #2 - Stack around the variable 'number_inp' was corrupted.

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

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

int main()
{
	char number_inp[4];
	short int num[4];
	short int i;
	
	scanf("%s", number_inp);
	for (i = 0; i < 4; i++)
		num[i] = number_inp[i] - '0';

	printf("%d", num[1]);
	return 0;

}

###試したこと
エラー内容を調べてみたところ領域を超えて確保してしまった、という内容でした。
書式文字列がいまいち理解できず、躓いています。
char型からint型にする際に符号拡張されてしまうという話も聞いてunsignedも試してみましたがよくわかりませんでした。
変換指定子や変数型の理解が足りてないならばそちらのキーワードを教えていただけるとありがたいです。

###補足情報(言語/FW/ツール等のバージョンなど)
より詳細な情報

Answer 1

scanf している箇所は4バイトのバッファを指定しています。ですので 123456 といった文字を入力するとバッファを超えて格納されます。おそらく NUL 分入れて5バイト必要ですね。
scanf が良く無い API とされている理由がこれです。
安全に行うには

char buf[256];
fgets(buf, sizeof(buf), stdin);
sscanf(buf, "%s", number_inp);

の様に十分なバッファで読み込ませるか

scanf("%3s", number_inp);

の様にサイズ指定されると良いかと思います。

追記

警告を消すには scanf_s を使って下さい。

scanf_s("%s", &number_inp, 3);

Comments

[Chironian]

scanf_sでC4996警告を消せますが、#define _CRT_SECURE_NO_WARNINGSでも消せますので、C4996は対処すみのようです。

[mattn]

ありがとうございます。そうですね、この場合はランタイムなので _CRT_SECURE_NO_WARNINGS は関係なく、ランタイムのオーバーフロー検出ですね。

[yowashi]

入力される際は、サイズを決めてあげるべきなんですね。納得しました。
その観点からすると、scanf( %3s)よりscanf_sで関数を作ったほうがいいんですかね。

[mattn]

scanf_s は標準関数ではないです。 _CRT_SECURE_NO_WARNINGS とあったので Windows と判断しましたので書かせて頂きました。ポータビリティを考えると書式対応が良いかと思います。

[majiponi]

"%4c"って書式設定なら、バッファ4バイトで、オーバーフローなしで入力できるよー。