Remove getmsg() on Solaris and use read() like everyone else.
Remove getmsg() on Solaris and use read() like everyone else.

As long as we put the file descriptor into "message-discard" mode with
the I_SRDOPT ioctl, it'll behave like every other OS and only return
one packet at a time even if more would fit in the buffer.

The MOREDATA flag doesn't mean what we thought it did, so getmsg() has no
efficiency advantages, and we're better off keeping it simple.

Also fix a bug introduced for script_tun mode by commit 337cf1a7.

Signed-off-by: David Woodhouse <b460d66aaf00c296a3db1c1d9eeafc081d5f7d70@intel.com>

nmav/openconnect-mine,mveplus/openconnect-client,liiir1985/openconnect,chadcatlett/openconnect,matlockx/openconnect,liiir1985/openconnect,liiir1985/openconnect,mtmiller/openconnect,matlockx/openconnect,mtmiller/openconnect,sihde/openconnect,don-johnny/openconnect,nmav/openconnect-mine,cernekee/openconnect,chadcatlett/openconnect,certik/openconnect,matlockx/openconnect,certik/openconnect,liiir1985/openconnect,cernekee/openconnect,mveplus/openconnect-client,sihde/openconnect,mdutton3/OpenConnect,atyndall/openconnect,sihde/openconnect,brandt/openconnect,neshema/openconnect,nmav/openconnect-mine,cernekee/openconnect,lins05/openconnect,lins05/openconnect,matlockx/openconnect,neshema/openconnect,don-johnny/openconnect,atyndall/openconnect,neshema/openconnect,chadcatlett/openconnect,chadcatlett/openconnect,don-johnny/openconnect,mtmiller/openconnect,brandt/openconnect,mtmiller/openconnect,brandt/openconnect,nmav/openconnect-mine,lins05/openconnect,lins05/openconnect,certik/openconnect,cernekee/openconnect,mdutton3/OpenConnect,neshema/openconnect,mveplus/openconnect-client,don-johnny/openconnect,atyndall/openconnect,mveplus/openconnect-client,brandt/openconnect,atyndall/openconnect,mdutton3/OpenConnect