请问这样编是错在哪了？应该是怎么编啊？

这是c语言练习题，我用这种循环的方法编的话是哪里错了呢，正确的方法应该是什么？

for循环部分改为如下即可

for(i=n;i>=1;i--)
   {
   if(m%i==0&&n%i==0){
       
       printf("%d",i);
       break;
   }
   }