The 5V protection looks O.K. Optimal resistor values may vary. Instead of clamping the 0-10V analog voltage to 10V, you can better clamp the divided voltage to 5V, saving the 10 V supply, provided it's not needed for other purposes.
You made a 11k/10k voltage divider which is probably not intended.
The general problem is that protection circuit has to be designed for a particular overvoltage level. No circuit can handle unlimited input voltage, you'll e.g. blow the resistors and diodes, or force the 5V supply. Smaller overvoltages can be safely handled by the ATmega substrate diodes in combination with a series resistor.