I am not sure what are you trying to do, but this part
Originally posted by raf
for rec_msg in my_cur
loop
messag := rec_msg.ename;
end loop;
will assign to "messag" only the last ename fetched.
I don't think you meant it this way.

Uff, even more... code after opening connection overwrites even this ename