AT89C51CC03 Taster problem

Hallo,
ich habe eine kleine frage zu einem kleinen Programm. Ich habe dort aber
ein kleines Problem am Ende. Undzwar will ich dort mit dem Taster S3
einmal eine Lampe Einschalten und bei einem nächsten Tastendruck wieder
ausschalten.
Dies funktioniert auch zu 80%, aber teilweise reagiert die LED nur
sporadisch. Schaltet also nicht sauber die zustände.
Hat da vllt. jemand einen Tipp?

Gruß

- - - Aktualisiert - - -

void main (void)
// Start des Hauptprogramms
{
// LED Fehler einschalten wenn S1 oder S2 gedrückt sind
// LED Grün = Alles OK
if ((S1==0) | (S2==0))
{
LED6 = AN;
LED7 = AUS;
}
else
{
LED7 = AN;
LED6 = AUS;
}
// Alarmanlage aktivieren

if((S3==0) & (S2==1) & (S1==1)) //S1 & S2 = Fenster-/Türkontakte
{ //S3 = Einschaltsensor
if(LED5==AUS)
LED5 = AN;

else
LED5 = AUS;
}
}