The program can not handle words longer than 7 characters. If you input a longer word it could lead to a program crash.
The %s specifier expects a char* but &s1 gives you a pointer to the array which has type char(*)[8]. I doubt it will lead to any trouble but to be correct it should be scanf("%s",&s1[0]); or scanf("%s",s1);