2003-03-06 Moritz Schulte <mo@g10code.com>
authorMoritz Schulte <mo@g10code.com>
Thu, 6 Mar 2003 21:55:44 +0000 (21:55 +0000)
committerMoritz Schulte <mo@g10code.com>
Thu, 6 Mar 2003 21:55:44 +0000 (21:55 +0000)
* secmem.h (GCRY_SECMEM_FLAG_NO_WARNING,
GCRY_SECMEM_FLAG_SUSPEND_WARNING): New symbols.

* global.c (gcry_control): Use
GCRY_SECMEM_FLAG_{NO,SUSPEND}_WARNING, instead of hard-coded
values.
* secmem.c (_gcry_secmem_set_flags): Likewise.
* secmem.c (_gcry_secmem_get_flags): Likewise.

src/ChangeLog
src/global.c
src/misc.c
src/secmem.c
src/secmem.h

index a59f2a2..00ff3c6 100644 (file)
@@ -1,3 +1,14 @@
+2003-03-06  Moritz Schulte  <mo@g10code.com>
+
+       * secmem.h (GCRY_SECMEM_FLAG_NO_WARNING,
+       GCRY_SECMEM_FLAG_SUSPEND_WARNING): New symbols.
+
+       * global.c (gcry_control): Use
+       GCRY_SECMEM_FLAG_{NO,SUSPEND}_WARNING, instead of hard-coded
+       values.
+       * secmem.c (_gcry_secmem_set_flags): Likewise.
+       * secmem.c (_gcry_secmem_get_flags): Likewise.
+       
 2003-03-03  Moritz Schulte  <moritz@g10code.com>
 
        * misc.c: Removed old FIXME, since there is already a function to
index f31f934..5d42e48 100644 (file)
@@ -193,15 +193,18 @@ gcry_control( enum gcry_global_control_cmds cmd, ... )
        break;
 
       case GCRYCTL_DISABLE_SECMEM_WARN:
-       _gcry_secmem_set_flags( (_gcry_secmem_get_flags() | 1) );
+       _gcry_secmem_set_flags ((_gcry_secmem_get_flags ()
+                                | GCRY_SECMEM_FLAG_NO_WARNING));
        break;
 
       case GCRYCTL_SUSPEND_SECMEM_WARN:
-       _gcry_secmem_set_flags( (_gcry_secmem_get_flags() | 2) );
+       _gcry_secmem_set_flags ((_gcry_secmem_get_flags ()
+                                | GCRY_SECMEM_FLAG_SUSPEND_WARNING));
        break;
 
       case GCRYCTL_RESUME_SECMEM_WARN:
-       _gcry_secmem_set_flags( (_gcry_secmem_get_flags() & ~2) );
+       _gcry_secmem_set_flags ((_gcry_secmem_get_flags ()
+                                & ~GCRY_SECMEM_FLAG_SUSPEND_WARNING));
        break;
 
       case GCRYCTL_USE_SECURE_RNDPOOL:
index b1a4064..12f37c0 100644 (file)
@@ -193,7 +193,7 @@ _gcry_log_fatal( const char *fmt, ... )
     va_start( arg_ptr, fmt ) ;
     _gcry_logv( GCRY_LOG_FATAL, fmt, arg_ptr );
     va_end(arg_ptr);
-    abort(); /* never called, bugs it makes the compiler happy */
+    abort(); /* never called, but it makes the compiler happy */
 }
 
 void
@@ -204,7 +204,7 @@ _gcry_log_bug( const char *fmt, ... )
     va_start( arg_ptr, fmt ) ;
     _gcry_logv( GCRY_LOG_BUG, fmt, arg_ptr );
     va_end(arg_ptr);
-    abort(); /* never called, bugs it makes the compiler happy */
+    abort(); /* never called, but it makes the compiler happy */
 }
 
 void
index bae42c3..7d0b681 100644 (file)
@@ -227,8 +227,8 @@ _gcry_secmem_set_flags( unsigned flags )
 {
     int was_susp = suspend_warning;
 
-    no_warning = flags & 1;
-    suspend_warning = flags & 2;
+    no_warning = flags & GCRY_SECMEM_FLAG_NO_WARNING;
+    suspend_warning = flags & GCRY_SECMEM_FLAG_SUSPEND_WARNING;
 
     /* and now issue the warning if it is not longer suspended */
     if( was_susp && !suspend_warning && show_warning ) {
@@ -242,8 +242,8 @@ _gcry_secmem_get_flags(void)
 {
     unsigned flags;
 
-    flags  = no_warning      ? 1:0;
-    flags |= suspend_warning ? 2:0;
+    flags  = no_warning ? GCRY_SECMEM_FLAG_NO_WARNING : 0;
+    flags |= suspend_warning ? GCRY_SECMEM_FLAG_SUSPEND_WARNING : 0;
     return flags;
 }
 
index 5c0a17b..553b385 100644 (file)
@@ -29,7 +29,10 @@ void _gcry_secmem_free (void *a);
 void _gcry_secmem_dump_stats (void);
 void _gcry_secmem_set_flags (unsigned flags);
 unsigned _gcry_secmem_get_flags(void) GCC_ATTR_PURE;
-
 int _gcry_private_is_secure (const void *p) GCC_ATTR_PURE;
 
+/* Flags for _gcry_secmem_{set,get}_flags.  */
+#define GCRY_SECMEM_FLAG_NO_WARNING      0x00000001
+#define GCRY_SECMEM_FLAG_SUSPEND_WARNING 0x00000002
+
 #endif /* G10_SECMEM_H */