Re: [PATCH V2 18/18] Tools: hv: Properly manage open streams

[Ben Hutchings]

On Mon, 2012-08-13 at 10:07 -0700, K. Y. Srinivasan wrote:
> Close the open streams properly.
> 
> Signed-off-by: K. Y. Srinivasan <kys@xxxxxxxxxxxxx>
> Reviewed-by: Haiyang Zhang <haiyangz@xxxxxxxxxxxxx>
> Reviewed-by: Olaf Hering <olaf@xxxxxxxxx>
> Reviewed-by: Ben Hutchings <ben@xxxxxxxxxxxxxxx>

This isn't reviewed by me, I'm the author!

Ben.

> ---
>  tools/hv/hv_kvp_daemon.c |    4 ++--
>  1 files changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/tools/hv/hv_kvp_daemon.c b/tools/hv/hv_kvp_daemon.c
> index c8e1013..ca6424f 100644
> --- a/tools/hv/hv_kvp_daemon.c
> +++ b/tools/hv/hv_kvp_daemon.c
> @@ -160,7 +160,7 @@ static void kvp_update_file(int pool)
>  				sizeof(struct kvp_record),
>  				kvp_file_info[pool].num_records, filep);
>  
> -	fflush(filep);
> +	fclose(filep);
>  	kvp_release_lock(pool);
>  }
>  
> @@ -206,7 +206,7 @@ static void kvp_update_mem_state(int pool)
>  	kvp_file_info[pool].num_blocks = num_blocks;
>  	kvp_file_info[pool].records = record;
>  	kvp_file_info[pool].num_records = records_read;
> -
> +	fclose(filep);
>  	kvp_release_lock(pool);
>  }
>  static int kvp_file_init(void)

-- 
Ben Hutchings
I say we take off; nuke the site from orbit.  It's the only way to be sure.

Attachment: signature.asc
Description: This is a digitally signed message part