aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorNIIBE Yutaka <[email protected]>2023-07-14 06:52:08 +0000
committerNIIBE Yutaka <[email protected]>2023-07-14 06:52:08 +0000
commitee9e3578ce30892ce86276819f0eedf8fb3f6e47 (patch)
tree7e3d77db4a7fbe497cba74373e1e63dd1a86d081
parentgpg: Use is_secured_filename before opening the file. (diff)
downloadgnupg-ee9e3578ce30892ce86276819f0eedf8fb3f6e47.tar.gz
gnupg-ee9e3578ce30892ce86276819f0eedf8fb3f6e47.zip
gpg: Use gnupg_fd_t for iobuf_get_fd and is_secured_file.
* common/iobuf.c (iobuf_get_fd): Return type is now gnupg_fd_t. * common/iobuf.h (iobuf_get_fd): Fix the return type. * g10/misc.c (is_secured_file): Argument is now gnupg_fd_t. * g10/main.h (is_secured_file): Fix the argument type. -- GnuPG-bug-id: 6580 Signed-off-by: NIIBE Yutaka <[email protected]>
-rw-r--r--common/iobuf.c6
-rw-r--r--common/iobuf.h2
-rw-r--r--g10/main.h2
-rw-r--r--g10/misc.c2
4 files changed, 6 insertions, 6 deletions
diff --git a/common/iobuf.c b/common/iobuf.c
index 3c5dc800c..f70ab9879 100644
--- a/common/iobuf.c
+++ b/common/iobuf.c
@@ -2641,20 +2641,20 @@ iobuf_get_filelength (iobuf_t a)
}
-int
+gnupg_fd_t
iobuf_get_fd (iobuf_t a)
{
for (; a->chain; a = a->chain)
;
if (a->filter != file_filter)
- return -1;
+ return GNUPG_INVALID_FD;
{
file_filter_ctx_t *b = a->filter_ov;
gnupg_fd_t fp = b->fp;
- return FD2INT (fp);
+ return fp;
}
}
diff --git a/common/iobuf.h b/common/iobuf.h
index 4a723908d..4354c718d 100644
--- a/common/iobuf.h
+++ b/common/iobuf.h
@@ -590,7 +590,7 @@ uint64_t iobuf_get_filelength (iobuf_t a);
/* Return the file descriptor designating the underlying file. This
only works with file_filter based pipelines. */
-int iobuf_get_fd (iobuf_t a);
+gnupg_fd_t iobuf_get_fd (iobuf_t a);
/* Return the real filename, if available. This only supports
pipelines that end in file filters. Returns NULL if not
diff --git a/g10/main.h b/g10/main.h
index 38e6a9c22..c202dacb8 100644
--- a/g10/main.h
+++ b/g10/main.h
@@ -107,7 +107,7 @@ char *make_radix64_string( const byte *data, size_t len );
void trap_unaligned(void);
void register_secured_file (const char *fname);
void unregister_secured_file (const char *fname);
-int is_secured_file (int fd);
+int is_secured_file (gnupg_fd_t fd);
int is_secured_filename (const char *fname);
u16 checksum_u16( unsigned n );
u16 checksum( const byte *p, unsigned n );
diff --git a/g10/misc.c b/g10/misc.c
index d1f0efc6e..ec9b9025d 100644
--- a/g10/misc.c
+++ b/g10/misc.c
@@ -160,7 +160,7 @@ unregister_secured_file (const char *fname)
/* Return true if FD is corresponds to a secured file. Using -1 for
FS is allowed and will return false. */
int
-is_secured_file (int fd)
+is_secured_file (gnupg_fd_t fd)
{
#ifdef ENABLE_SELINUX_HACKS
struct stat buf;