nbd-server: fix setup_serve() to report errors
authorTuomas Räsänen <tuomasjjrasanen@opinsys.fi>
Sat, 12 Jan 2013 21:25:12 +0000 (23:25 +0200)
committerWouter Verhelst <w@uter.be>
Sat, 19 Jan 2013 10:11:40 +0000 (11:11 +0100)
commit428146730858495b37ea7bb98ba92b31903df11b
treed3b1f653fce2e36b74742c629321534fe016fb77
parent4476b001cea5a169c9eff0f3409d4720ee8cd623
nbd-server: fix setup_serve() to report errors

This commit changes setup_serve() to report its errors via GError object
to its caller. It also add resource deallocation code to return points
to free all local resources correctly. Note that, the previous version
of the setup_serve() did not free its resources properly on error, but
just called functions which terminated the process. This was techincally
a leak, but without any practical significance due to immediate process
termination. However, now that error handling (process termination) is
separeted from error reporting, we must deallocate/free all locally
allocated resources correctly on errors.

Signed-off-by: Tuomas Räsänen <tuomasjjrasanen@tjjr.fi>
nbd-server.c