[Freeipa-devel] [PATCH] add configure check for errno_t

Sumit Bose sbose at redhat.com
Fri Jul 24 21:05:45 UTC 2009


Hi,

we regular use functions returning errno error code like ENOMEN or
EINVAL. The return type of these functions is usually int. To underline
that the return code is only an error code and should not be used
otherwise a special type can be used. There is the ISO/ICE technical
report 24731-1 which recommends they type errno_t for this purpose. I
think it makes sense to stick with this recommendation (and I hope
people don't mind that the same TR also recommends the not so smart *_s
replacements for most of the standard libc functions).

The attached patch add a configure check for errno_t and defines it if
not present in the system.

bye,
Sumit
-------------- next part --------------
>From ad5a2894e9404a3fba0f5f94a3a01b7f42b8df82 Mon Sep 17 00:00:00 2001
From: Sumit Bose <sbose at redhat.com>
Date: Fri, 24 Jul 2009 22:41:17 +0200
Subject: [PATCH] add configure check for errno_t

---
 server/configure.ac |    2 ++
 server/util/util.h  |    4 ++++
 2 files changed, 6 insertions(+), 0 deletions(-)

diff --git a/server/configure.ac b/server/configure.ac
index 5a9888b..d9e1e0e 100644
--- a/server/configure.ac
+++ b/server/configure.ac
@@ -31,6 +31,8 @@ AM_CONDITIONAL([HAVE_GCC], [test "$ac_cv_prog_gcc" = yes])
 AC_CHECK_HEADERS(stdint.h dlfcn.h)
 AC_CONFIG_HEADER(config.h)
 
+AC_CHECK_TYPES([errno_t], [], [], [[#include <errno.h>]])
+
 TESTS=""
 EXTRA_OBJ=""
 
diff --git a/server/util/util.h b/server/util/util.h
index 02916c1..b60dfa8 100644
--- a/server/util/util.h
+++ b/server/util/util.h
@@ -14,6 +14,10 @@
 #include "tevent.h"
 #include "ldb.h"
 
+#ifndef HAVE_ERRNO_T
+typedef int errno_t;
+#endif
+
 extern const char *debug_prg_name;
 extern int debug_level;
 extern int debug_timestamps;
-- 
1.6.2.5



More information about the Freeipa-devel mailing list