When failure occurs, 'ret' need be set, or may return 0 to indicate
success. Previously, an error was set in errp, but 0 was returned
anyway. So let bdrv_append_temp_snapshot() return an error code and
use that for the bdrv_open() return value.

Also, error_propagate() need be called only one time within a function.
Signed-off-by: NChen Gang <gang.chen.5i5j@gmail.com>
Signed-off-by: NKevin Wolf <kwolf@redhat.com>