No, mettere buffer[bytes - 1] perchè gli array in C iniziano da zero, metterei anche un controllo sul fatto che la read abbia letto qualcosa (se no fa del casino incredibile).
Per il fatto del 0xFF, anche se buffer è un'array di char, non devi per forza passagli un char con le '' (puoi usando le sequenze di escape), puoi anche passare direttamente il valore:
codice:
buffer[bytes] == 0xFF
Di più non puoi perchè il char è un 8bit.