Сбой Arduino и перезагрузка в случайных точках

Вероятно, у вас мало оперативной памяти.

Кроме того, вы не должны использовать класс String для firstLine. Это вызывает множество проблем, некоторые из которых могут возникать в случайное время.

Просто используйте массив символов, как вы сделали для received. Сохраняйте каждый символ в массиве до тех пор, пока не появится символ новой строки, и увеличивайте длину для каждого символа:

void checkForClient(int loading){
  char    firstLine[60];
  uint8_t firstLineLen = 0;

  MP3player.pauseDataStream ();
  EthernetClient client = server.available();
  MP3player.resumeDataStream();
  if (client) {
    if(!loading){
      Serial.println(F("new client")); // CRASH HERE
      bool endLn(false);
      char chr;
      int i(0);
      while (clientConnected(client)) {
        char received[clientAvailable(client)+2];
        while (clientAvailable(client)) {
          MP3player.pauseDataStream ();
          char c = client.read();
          MP3player.resumeDataStream ();
          received[i] = c;
          Serial.print(c); // OR HERE
          if (!endLn) {
            if (firstLineLength < sizeof(firstLine)-2)
              firstLine[ firstLineLength++ ] = c;
            if (c == '\n') {
              endLn = true;
              firstLine[ firstLineLength ] = '\0'; // NUL-terminate
            }
          }
          ++i;
        }
        received[i+1] = '\0';
        i = 0;
        if(endLn){
          Serial.println();
          endLn = false;
          Serial.print(F("first line : "));
          Serial.println(firstLine);
        }

Это также сэкономит около 1600 байт программного пространства. Есть много причин избегать String, и многие люди писали о подводных камнях. Если вы используете String в другом месте своей программы, вы должны использовать аналогичный подход для его устранения.

Еще один способ уменьшить объем оперативной памяти — избегать сохранения данных при их поступлении, а затем их последующего использования. Вместо этого используйте его немедленно:

void checkForClient(int loading){

  MP3player.pauseDataStream ();
  EthernetClient client = server.available();
  MP3player.resumeDataStream();
  if (client) {
    if(!loading){
      Serial.println(F("new client")); // CRASH HERE

      Serial.print(F("first line : "));
      bool endLn(false);

      char chr;
      int i(0);
      while (clientConnected(client)) {
        char received[clientAvailable(client)+2];
        while (clientAvailable(client)) {
          MP3player.pauseDataStream ();
          char c = client.read();
          MP3player.resumeDataStream ();
          received[i] = c;
          if (!endLn) {
            Serial.print(c); // printed now instead of saved for later
            if (c == '\n') {
              endLn = true;
            }
          }
          ++i;
        }
        received[i+1] = '\0';
        i = 0;
        if (endLn) {
          Serial.println();
          endLn = false;
          //Serial.println(firstLine); // NOT NEEDED, already printed!
        }

Это экономит всю оперативную память, которую вы бы использовали для firstLine.

Вы не разместили всю свою программу для нашего обзора, поэтому вам придется искать другие переменные, которые слишком велики или могут быть объявлены внутри подпрограммы (т. е. локальной переменной) вместо этого. в области файла (т. е. глобальная переменная, объявленная вне каких-либо подпрограмм).