Buffer overflow detected in a program that runs flawlessly ( apparently)

Question

On a string containing as, bs and cs, we can perform the following operation. We can take any two adjacent different characters and replace them by the third. For example, 'ab' gets reduced to 'c' and so does 'ba' and so on. I wrote this code to perform the following operation on t strings (t<=100) and max length of strings = 100 ``` #include #include #include int redlen(char string[100][100], int x) { int g, checker; checker = 1; for(; checker; ) { checker = 0; for(int i = 0;string[x][i]!='\0'; i++) { if((string[x][i]=='a' && string[x][i+1]=='b') || (string[x][i]=='b' && string[x][i+1]=='a')) { string[x][i]='c'; checker = 1; for(g = i+1; string[x][g]!='\0'; g++) { string[x][g]=string[x][g+1]; } i = 0; } else if((string[x][i]=='b' && string[x][i+1]=='c') || (string[x][i]=='c' && string[x][i+1]=='b')) { string[x][i]='a'; checker = 1; for(g = i+1; string[x][g]!='\0'; g++) { string[x][g]=string[x][g+1]; } i = 0; } else if((string[x][i]=='a' && string[x][i+1]=…

Accepted Answer

```
   int t; char r[3];
   gets(r);
```

Never ever use `gets` function. In this case if it receives more than `3` characters (including the terminating null character) you get a buffer overflow.

`gets` has been removed in the current Standard (C11) and was deprecated in the previous Standard (C99).

Buffer overflow detected in a program that runs flawlessly ( apparently)

Solution

Problem