> > + assert(vs); > > return vnc_socket_local_addr("%s:%s", vs->lsock); > > } > > > > > > If you want, I can just silence the warning in Coverity.
I think in the source code is better, human readers might wonder too. And in case the calling code ever changes behavior the assert will trap it (even though it is highly unlikely in this specific case). cheers, Gerd