vmdk: fix vmdk_parse_extents() extent_file leaks
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com> Reviewed-by: Max Reitz <mreitz@redhat.com> Reviewed-by: Fam Zheng <famz@redhat.com>
This commit is contained in:
		
							parent
							
								
									c5fe97e359
								
							
						
					
					
						commit
						ff74f33c31
					
				| @ -834,6 +834,7 @@ static int vmdk_parse_extents(const char *desc, BlockDriverState *bs, | |||||||
|             ret = vmdk_add_extent(bs, extent_file, true, sectors, |             ret = vmdk_add_extent(bs, extent_file, true, sectors, | ||||||
|                             0, 0, 0, 0, 0, &extent, errp); |                             0, 0, 0, 0, 0, &extent, errp); | ||||||
|             if (ret < 0) { |             if (ret < 0) { | ||||||
|  |                 bdrv_unref(extent_file); | ||||||
|                 return ret; |                 return ret; | ||||||
|             } |             } | ||||||
|             extent->flat_start_offset = flat_offset << 9; |             extent->flat_start_offset = flat_offset << 9; | ||||||
| @ -853,6 +854,7 @@ static int vmdk_parse_extents(const char *desc, BlockDriverState *bs, | |||||||
|             extent = &s->extents[s->num_extents - 1]; |             extent = &s->extents[s->num_extents - 1]; | ||||||
|         } else { |         } else { | ||||||
|             error_setg(errp, "Unsupported extent type '%s'", type); |             error_setg(errp, "Unsupported extent type '%s'", type); | ||||||
|  |             bdrv_unref(extent_file); | ||||||
|             return -ENOTSUP; |             return -ENOTSUP; | ||||||
|         } |         } | ||||||
|         extent->type = g_strdup(type); |         extent->type = g_strdup(type); | ||||||
|  | |||||||
		Loading…
	
	
			
			x
			
			
		
	
		Reference in New Issue
	
	Block a user
	 Stefan Hajnoczi
						Stefan Hajnoczi