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 <[email protected]> > --- > qemu-img.c | 1 + > 1 file changed, 1 insertion(+) Reviewed-by: Max Reitz <[email protected]>
signature.asc
Description: OpenPGP digital signature
