On 19.10.18 22:39, Liam Merwick wrote: > Although the function block_job_get() can return NULL, it would be a > serious bug if it did so (because the job yields before executing anything > (if it started successfully); but otherwise, commit_active_start() would > have returned an error). However, as a precaution, before dereferencing > the 'job' pointer in img_commit() assert it is not NULL.
(In the meantime, Markus has argued to me in some other case that asserting that something isn't NULL is just as good as just dereferencing it. Oh well, I still don't mind either way.) > Signed-off-by: Liam Merwick <liam.merw...@oracle.com> > --- > qemu-img.c | 1 + > 1 file changed, 1 insertion(+) Reviewed-by: Max Reitz <mre...@redhat.com>
signature.asc
Description: OpenPGP digital signature
