既にクローズされていますので参考までに。

gcc には `sanitize` オプションというものがありまして、その内の一つに配列変数などの宣言領域を超えるアクセス(`out-of-bounds access`)を実行時に検出する機能があります。具体的には `-fsanitize=address` を付けてコンパイルします。

    $ gcc -std=gnu99 -Wall -fsanitize=address -g cloth.c -o cloth
    $ ./cloth
    4
    =================================================================
    ==26858==ERROR: AddressSanitizer: stack-buffer-overflow on address 0xbfe95522 ...
    WRITE of size 3 at 0xbfe95522 thread T0
        #0 0xb71e211b  (/usr/lib/i386-linux-gnu/libasan.so.3+0x4311b)
        #1 0xb71e2cab in __isoc99_vscanf (/usr/lib/i386-linux-gnu/libasan.so.3+0x43cab)
        #2 0xb71e2d27 in __interceptor___isoc99_scanf (...
        #3 0x804892f in main cloth.c:26  <== "int scanf_count = scanf("%2s", card);"
        #4 0xb7001636 in __libc_start_main (/lib/i386-linux-gnu/libc.so.6+0x18636)
        #5 0x8048780  (cloth+0x8048780)
    
    Address 0xbfe95522 is located in stack of thread T0 at offset 194 in frame
        #0 0x804886a in main cloth.c:20  <== "main function frame"
    
      This frame has 3 object(s):
        [32, 136) 'player_list'
        [192, 194) 'card' <== Memory access at offset 194 overflows this variable
        [256, 270) 'card_left'

char card[3]; に修正すると、上記のエラーメッセージは出力されなくなります。

ところで、`scanf(3)` の書式文字列には `POSIX.1-2008` において導入された `m modifier` を使うことができます。この場合、配列変数を用意する必要がなくなります。ただ、下記に引用した文章に書かれてある通り、確保したメモリ領域を `free(3)` で解放する必要はあります。

**scanf(3)**

> **DESCRIPTION**
>
> An optional 'm' character. This is used with string conversions (%s, %c, %[), and relieves the caller of the need to allocate a corresponding buffer to hold the input: instead, scanf() allocates a buffer of sufficient size, and assigns the address of this buffer to the corresponding pointer argument, which should be a pointer to a char * variable (this variable does not need to be initialized before the call). The caller should subsequently free(3) this buffer when it is no longer required.
>
>     char *buf;
>     scanf("%ms", &buf);
>           :
>     free(buf);