if (digitalRead(button1) == LOW && led1 == HIGH)
You are testing the value of the led pin not the state of the led pin