Ein sei(); gehört NICHT in eine ISR. Da sorgt der Compiler selbst dafür, dass nichts passiert. Da bringt man den Controller eher durcheinander, da der Interrupt dann z.b. schon angesprungen werden kann, während er noch läuft => Problem.

Zwar kein Fehler aber unnötigt: Du initialisiert einiges doppelt; Z.b. sei() reicht 1x in der Main, ebenso DDRD.