Are you working to a circuit diagram or making it up as you go?

Be careful with that central bolt on the transformer. Have a look at the Printed Circuit board underneath. Does the bolt connect to a copper plane at all? If it does you should attach your earth to that side not the top or better, dont attach to that bolt at all, but use metal mounting posts for the PCB to earth it to the chassis. If it doesn't, i.e. its insulated from the circuit board, then you don't need an earth there at all. The way you have it wired could be a problem if the bolt should touch the chassis, or if its connected to something on the circuit board that could end up connecting to the chassis, because then what you will have created is a shorted turn on the transformer which will at best give some loss of power and at worst could cause overheating and melt things...

Click image for larger version. 

Name:	toroid gotcha.gif 
Views:	444 
Size:	28.8 KB 
ID:	7087