Compare string of the serial interface using strcmp and Serial.read()

sorry...
is this correct way ?
i dont have access to arduino at the moment.

if (Serial1.available() > 0) 
  {             
       while (Serial1.available() > 0) {
         inSerial[i]=Serial1.read(); //read data  
         i++;      
       }
       if(inSerial[i]=='\r\n'){  
          Check_Protocol(inSerial);
       }else{
       inSerial[i]='\0';
      Check_Protocol(inSerial);
      }
    }