Is simple XOR with secret key using DHE secure?
In addition to the problems that user595228 has mentioned, well, an attacker can easily solve a discrete log modulo a 256 bit prime; from that, he can recompute the shared secret, and that would give him the entire message.  To be secure, you really need at least a 1024 bit prime; with a 2048 bit prime being greatly preferred.