Re: [PATCH v6 1/2] block: Ignore close() failure in get_tmp_filename()

[Kevin Wolf]

Am 10.10.2022 um 06:04 hat Bin Meng geschrieben:
> The temporary file has been created and is ready for use. Checking
> return value of close() does not seem useful. The file descriptor
> is almost certainly closed; see close(2) under "Dealing with error
> returns from close()".
> 
> Let's simply ignore close() failure here.
> 
> Suggested-by: Markus Armbruster 
> Signed-off-by: Bin Meng 
> Reviewed-by: Markus Armbruster 

Thanks, series applied to the block branch.

Kevin

[PATCH v6 1/2] block: Ignore close() failure in get_tmp_filename()

[Bin Meng]

The temporary file has been created and is ready for use. Checking
return value of close() does not seem useful. The file descriptor
is almost certainly closed; see close(2) under "Dealing with error
returns from close()".

Let's simply ignore close() failure here.

Suggested-by: Markus Armbruster 
Signed-off-by: Bin Meng 
Reviewed-by: Markus Armbruster 
---

(no changes since v5)

Changes in v5:
- new patch: "block: Ignore close() failure in get_tmp_filename()"

 block.c | 5 +
 1 file changed, 1 insertion(+), 4 deletions(-)

diff --git a/block.c b/block.c
index bc85f46eed..582c205307 100644
--- a/block.c
+++ b/block.c
@@ -886,10 +886,7 @@ int get_tmp_filename(char *filename, int size)
 if (fd < 0) {
 return -errno;
 }
-if (close(fd) != 0) {
-unlink(filename);
-return -errno;
-}
+close(fd);
 return 0;
 #endif
 }
-- 
2.25.1