From d68a803c4760415204fb3e7af4c0195ee2cd09ad Mon Sep 17 00:00:00 2001 From: NIIBE Yutaka Date: Fri, 7 Oct 2022 11:16:55 +0900 Subject: gpg: Fix wrong use of FD2INT with iobuf_fdopen_nc. * g10/decrypt.c (decrypt_message_fd): Use INPUT_FD directly. * g10/encrypt.c (encrypt_crypt): Use FILEFD directly. -- Before 8402815d, original code was with iobuf_open_fd_or_name, which used gnupg_fd_t for the file descriptor (FD2INT was relevant at that time). After the change, because it's not gnupg_fd_t but int, use of FD2INT is irrelevant. Fixes-commit: 8402815d8e0e04a44362968f88b3d484d2395402 Signed-off-by: NIIBE Yutaka --- g10/decrypt.c | 2 +- g10/encrypt.c | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/g10/decrypt.c b/g10/decrypt.c index 065601b7b..cb9e36a93 100644 --- a/g10/decrypt.c +++ b/g10/decrypt.c @@ -120,7 +120,7 @@ decrypt_message_fd (ctrl_t ctrl, int input_fd, int output_fd) pfx = new_progress_context (); /* Open the message file. */ - fp = iobuf_fdopen_nc (FD2INT(input_fd), "rb"); + fp = iobuf_fdopen_nc (input_fd, "rb"); if (fp && is_secured_file (iobuf_get_fd (fp))) { iobuf_close (fp); diff --git a/g10/encrypt.c b/g10/encrypt.c index dadb726b3..019bf0be4 100644 --- a/g10/encrypt.c +++ b/g10/encrypt.c @@ -829,10 +829,10 @@ encrypt_crypt (ctrl_t ctrl, int filefd, const char *filename, gpg_err_set_errno (ENOSYS); } #else - if (filefd == GNUPG_INVALID_FD) + if (filefd == -1) inp = iobuf_open (filename); else - inp = iobuf_fdopen_nc (FD2INT(filefd), "rb"); + inp = iobuf_fdopen_nc (filefd, "rb"); #endif if (inp) iobuf_ioctl (inp, IOBUF_IOCTL_NO_CACHE, 1, NULL); -- cgit v1.2.3