]> git.sur5r.net Git - openldap/blobdiff - libraries/liblber/io.c
Remove lint
[openldap] / libraries / liblber / io.c
index 31e872c647af420cdfa50f842afbf807adbccea4..05773e900bf407bcde702dd17262311dbdd57db5 100644 (file)
@@ -1,6 +1,7 @@
 /* io.c - ber general i/o routines */
+/* $OpenLDAP$ */
 /*
- * Copyright 1998-1999 The OpenLDAP Foundation, All Rights Reserved.
+ * Copyright 1998-2000 The OpenLDAP Foundation, All Rights Reserved.
  * COPYING RESTRICTIONS APPLY, see COPYRIGHT file
  */
 /* Portions
@@ -18,7 +19,8 @@
 #include "portable.h"
 
 #include <stdio.h>
-#include <stdlib.h>
+
+#include <ac/stdlib.h>
 
 #include <ac/ctype.h>
 #include <ac/errno.h>
@@ -32,9 +34,6 @@
 
 #include "lber-int.h"
 
-static long BerRead LDAP_P(( Sockbuf *sb, char *buf, long len ));
-static int ber_realloc LDAP_P(( BerElement *ber, unsigned long len ));
-
 #define EXBUFSIZ       1024
 
 /* probably far too large... */
@@ -45,22 +44,27 @@ static int ber_realloc LDAP_P(( BerElement *ber, unsigned long len ));
 # define MAX_BERBUFSIZE 65535
 #endif
 
-static long
-BerRead( Sockbuf *sb, char *buf, long len )
+static ber_slen_t
+BerRead(
+       Sockbuf *sb,
+       unsigned char *buf,
+       ber_len_t len )
 {
-       int     c;
-       long    nread = 0;
+       ber_slen_t      c;
+       ber_slen_t      nread = 0;
 
        assert( sb != NULL );
        assert( buf != NULL );
 
+       assert( SOCKBUF_VALID( sb ) );
+
        while ( len > 0 ) {
-               if ( (c = ber_pvt_sb_read( sb, buf, len )) <= 0 ) {
+               if ( (c = ber_int_sb_read( sb, buf, len )) <= 0 ) {
                        if ( nread > 0 )
                                break;
                        return( c );
                }
-               buf+= c;
+               buf+=c;
                nread+=c;
                len-=c;
        }
@@ -68,34 +72,41 @@ BerRead( Sockbuf *sb, char *buf, long len )
        return( nread );
 }
 
-long
-ber_read( BerElement *ber, char *buf, unsigned long len )
+ber_slen_t
+ber_read(
+       BerElement *ber,
+       char *buf,
+       ber_len_t len )
 {
-       unsigned long   actuallen, nleft;
+       ber_len_t       actuallen, nleft;
 
        assert( ber != NULL );
        assert( buf != NULL );
 
+       assert( BER_VALID( ber ) );
+
        nleft = ber->ber_end - ber->ber_ptr;
        actuallen = nleft < len ? nleft : len;
 
-       SAFEMEMCPY( buf, ber->ber_ptr, (size_t)actuallen );
+       SAFEMEMCPY( buf, ber->ber_ptr, actuallen );
 
        ber->ber_ptr += actuallen;
 
-       return( (long)actuallen );
+       return( (ber_slen_t) actuallen );
 }
 
-long
+ber_slen_t
 ber_write(
        BerElement *ber,
        LDAP_CONST char *buf,
-       unsigned long len,
+       ber_len_t len,
        int nosos )
 {
        assert( ber != NULL );
        assert( buf != NULL );
 
+       assert( BER_VALID( ber ) );
+
        if ( nosos || ber->ber_sos == NULL ) {
                if ( ber->ber_ptr + len > ber->ber_end ) {
                        if ( ber_realloc( ber, len ) != 0 )
@@ -103,7 +114,8 @@ ber_write(
                }
                SAFEMEMCPY( ber->ber_ptr, buf, (size_t)len );
                ber->ber_ptr += len;
-               return( len );
+               return( (ber_slen_t) len );
+
        } else {
                if ( ber->ber_sos->sos_ptr + len > ber->ber_end ) {
                        if ( ber_realloc( ber, len ) != 0 )
@@ -112,14 +124,14 @@ ber_write(
                SAFEMEMCPY( ber->ber_sos->sos_ptr, buf, (size_t)len );
                ber->ber_sos->sos_ptr += len;
                ber->ber_sos->sos_clen += len;
-               return( len );
+               return( (ber_slen_t) len );
        }
 }
 
-static int
-ber_realloc( BerElement *ber, unsigned long len )
+int
+ber_realloc( BerElement *ber, ber_len_t len )
 {
-       unsigned long   need, have, total;
+       ber_len_t       need, have, total;
        Seqorset        *s;
        long            off;
        char            *oldbuf;
@@ -127,18 +139,20 @@ ber_realloc( BerElement *ber, unsigned long len )
        assert( ber != NULL );
        assert( len > 0 );
 
+       assert( BER_VALID( ber ) );
+
        have = (ber->ber_end - ber->ber_buf) / EXBUFSIZ;
        need = (len < EXBUFSIZ ? 1 : (len + (EXBUFSIZ - 1)) / EXBUFSIZ);
        total = have * EXBUFSIZ + need * EXBUFSIZ;
 
        oldbuf = ber->ber_buf;
 
+       ber->ber_buf = (char *) LBER_REALLOC( ber->ber_buf, total );
+       
        if ( ber->ber_buf == NULL ) {
-               if ( (ber->ber_buf = (char *) malloc( (size_t)total )) == NULL )
-                       return( -1 );
-       } else if ( (ber->ber_buf = (char *) realloc( ber->ber_buf,
-           (size_t)total )) == NULL )
+               ber->ber_buf = oldbuf;
                return( -1 );
+       }
 
        ber->ber_end = ber->ber_buf + total;
 
@@ -151,7 +165,7 @@ ber_realloc( BerElement *ber, unsigned long len )
        if ( ber->ber_buf != oldbuf ) {
                ber->ber_ptr = ber->ber_buf + (ber->ber_ptr - oldbuf);
 
-               for ( s = ber->ber_sos; s != NULLSEQORSET; s = s->sos_next ) {
+               for ( s = ber->ber_sos; s != NULL; s = s->sos_next ) {
                        off = s->sos_first - oldbuf;
                        s->sos_first = ber->ber_buf + off;
 
@@ -166,22 +180,45 @@ ber_realloc( BerElement *ber, unsigned long len )
 void
 ber_free( BerElement *ber, int freebuf )
 {
+#ifdef LDAP_MEMORY_DEBUG
        assert( ber != NULL );
+#endif
+
+       if( ber == NULL ) {
+               return;
+       }
+
+       assert( BER_VALID( ber ) );
+
+       if ( freebuf ) {
+               Seqorset *s, *next;
+               LBER_FREE( ber->ber_buf );
+
+               for( s = ber->ber_sos ; s != NULL ; s = next ) {
+                       next = s->sos_next;
+                       LBER_FREE( s );
+               }
+       }
 
-       if ( freebuf && ber->ber_buf != NULL )
-               free( ber->ber_buf );
        ber->ber_buf = NULL;
-       free( (char *) ber );
+       ber->ber_sos = NULL;
+       ber->ber_valid = LBER_UNINITIALIZED;
+
+       LBER_FREE( (char *) ber );
 }
 
 int
 ber_flush( Sockbuf *sb, BerElement *ber, int freeit )
 {
-       long    nwritten, towrite, rc;  
+       ber_len_t       nwritten, towrite;
+       ber_slen_t      rc;     
 
        assert( sb != NULL );
        assert( ber != NULL );
 
+       assert( SOCKBUF_VALID( sb ) );
+       assert( BER_VALID( ber ) );
+
        if ( ber->ber_rwptr == NULL ) {
                ber->ber_rwptr = ber->ber_buf;
        }
@@ -189,25 +226,16 @@ ber_flush( Sockbuf *sb, BerElement *ber, int freeit )
 
        if ( sb->sb_debug ) {
                ber_log_printf( LDAP_DEBUG_ANY, sb->sb_debug,
-                       "ber_flush: %ld bytes to sd %ld%s\n", towrite,
-                   (long) sb->sb_sd, ber->ber_rwptr != ber->ber_buf ? " (re-flush)"
-                   : "" );
+                       "ber_flush: %ld bytes to sd %ld%s\n",
+                       towrite, (long) sb->sb_fd,
+                       ber->ber_rwptr != ber->ber_buf ?  " (re-flush)" : "" );
                ber_log_bprint( LDAP_DEBUG_PACKETS, sb->sb_debug,
                        ber->ber_rwptr, towrite );
        }
 
-#if !defined(MACOS) && !defined(DOS)
-       if ( sb->sb_options & (LBER_TO_FILE | LBER_TO_FILE_ONLY) ) {
-               rc = write( sb->sb_fd, ber->ber_rwptr, towrite );
-               if ( sb->sb_options & LBER_TO_FILE_ONLY ) {
-                       return( (int)rc );
-               }
-       }
-#endif
-       
        nwritten = 0;
        do {
-               rc = ber_pvt_sb_write( sb, ber->ber_rwptr, towrite );
+               rc = ber_int_sb_write( sb, ber->ber_rwptr, towrite );
                if (rc<=0) {
                        return -1;
                }
@@ -227,42 +255,49 @@ ber_alloc_t( int options )
 {
        BerElement      *ber;
 
-       ber = (BerElement *) calloc( 1, sizeof(BerElement) );
+    ber_int_options.lbo_valid = LBER_INITIALIZED;
+
+       ber = (BerElement *) LBER_CALLOC( 1, sizeof(BerElement) );
 
-       if ( ber == NULLBER )
-               return( NULLBER );
+       if ( ber == NULL )
+               return( NULL );
 
+       ber->ber_valid = LBER_VALID_BERELEMENT;
        ber->ber_tag = LBER_DEFAULT;
        ber->ber_options = options;
        ber->ber_debug = ber_int_debug;
 
+       assert( BER_VALID( ber ) );
        return( ber );
 }
 
 BerElement *
-ber_alloc( void )
+ber_alloc( void )      /* deprecated */
 {
        return( ber_alloc_t( 0 ) );
 }
 
 BerElement *
-der_alloc( void )
+der_alloc( void )      /* deprecated */
 {
        return( ber_alloc_t( LBER_USE_DER ) );
 }
 
 BerElement *
-ber_dup( LDAP_CONST BerElement *ber )
+ber_dup( BerElement *ber )
 {
        BerElement      *new;
 
        assert( ber != NULL );
+       assert( BER_VALID( ber ) );
 
-       if ( (new = ber_alloc()) == NULL )
+       if ( (new = ber_alloc_t( ber->ber_options )) == NULL ) {
                return( NULL );
+       }
 
        *new = *ber;
 
+       assert( BER_VALID( new ) );
        return( new );
 }
 
@@ -273,9 +308,15 @@ ber_init_w_nullc( BerElement *ber, int options )
 {
        assert( ber != NULL );
 
+    ber_int_options.lbo_valid = LBER_INITIALIZED;
+
        (void) memset( (char *)ber, '\0', sizeof( BerElement ));
+       ber->ber_valid = LBER_VALID_BERELEMENT;
        ber->ber_tag = LBER_DEFAULT;
        ber->ber_options = (char) options;
+       ber->ber_debug = ber_int_debug;
+
+       assert( BER_VALID( ber ) );
 }
 
 /* New C-API ber_init() */
@@ -289,19 +330,21 @@ ber_init( struct berval *bv )
 
        assert( bv != NULL );
 
+    ber_int_options.lbo_valid = LBER_INITIALIZED;
+
        if ( bv == NULL ) {
                return NULL;
        }
 
        ber = ber_alloc_t( 0 );
 
-       if( ber == NULLBER ) {
+       if( ber == NULL ) {
                /* allocation failed */
                return ( NULL );
        }
 
        /* copy the data */
-       if ( ( (unsigned int) ber_write ( ber, bv->bv_val, bv->bv_len, 0 )) != bv->bv_len ) {
+       if ( ( (ber_len_t) ber_write ( ber, bv->bv_val, bv->bv_len, 0 )) != bv->bv_len ) {
                /* write failed, so free and return NULL */
                ber_free( ber, 1 );
                return( NULL );
@@ -318,18 +361,20 @@ ber_init( struct berval *bv )
 ** the returned berval.
 */
 int ber_flatten(
-       LDAP_CONST BerElement *ber,
+       BerElement *ber,
        struct berval **bvPtr)
 {
        struct berval *bv;
  
        assert( bvPtr != NULL );
 
+    ber_int_options.lbo_valid = LBER_INITIALIZED;
+
        if(bvPtr == NULL) {
                return( -1 );
        }
 
-       if ( (bv = malloc( sizeof(struct berval))) == NULL ) {
+       if ( (bv = LBER_MALLOC( sizeof(struct berval))) == NULL ) {
                return( -1 );
        }
 
@@ -340,14 +385,14 @@ int ber_flatten(
 
        } else {
                /* copy the berval */
-               ptrdiff_t len = ber->ber_ptr - ber->ber_buf;
+               ber_len_t len = ber->ber_ptr - ber->ber_buf;
 
-               if ( (bv->bv_val = (char *) malloc( len + 1 )) == NULL ) {
+               if ( (bv->bv_val = (char *) LBER_MALLOC( len + 1 )) == NULL ) {
                        ber_bvfree( bv );
                        return( -1 );
                }
 
-               SAFEMEMCPY( bv->bv_val, ber->ber_buf, (size_t)len );
+               SAFEMEMCPY( bv->bv_val, ber->ber_buf, len );
                bv->bv_val[len] = '\0';
                bv->bv_len = len;
        }
@@ -360,6 +405,7 @@ void
 ber_reset( BerElement *ber, int was_writing )
 {
        assert( ber != NULL );
+       assert( BER_VALID( ber ) );
 
        if ( was_writing ) {
                ber->ber_end = ber->ber_ptr;
@@ -373,26 +419,27 @@ ber_reset( BerElement *ber, int was_writing )
 
 #if 0
 /* return the tag - LBER_DEFAULT returned means trouble */
-static unsigned long
+static ber_tag_t
 get_tag( Sockbuf *sb )
 {
        unsigned char   xbyte;
-       unsigned long   tag;
+       ber_tag_t       tag;
        char            *tagp;
        unsigned int    i;
 
        assert( sb != NULL );
+       assert( SOCKBUF_VALID( sb ) );
 
-       if ( ber_pvt_sb_read( sb, (char *) &xbyte, 1 ) != 1 )
+       if ( ber_int_sb_read( sb, (char *) &xbyte, 1 ) != 1 )
                return( LBER_DEFAULT );
 
        if ( (xbyte & LBER_BIG_TAG_MASK) != LBER_BIG_TAG_MASK )
-               return( (unsigned long) xbyte );
+               return( (ber_tag_t) xbyte );
 
        tagp = (char *) &tag;
        tagp[0] = xbyte;
-       for ( i = 1; i < sizeof(long); i++ ) {
-               if ( ber_pvt_sb_read( sb, (char *) &xbyte, 1 ) != 1 )
+       for ( i = 1; i < sizeof(ber_tag_t); i++ ) {
+               if ( ber_int_sb_read( sb, (char *) &xbyte, 1 ) != 1 )
                        return( LBER_DEFAULT );
 
                tagp[i] = xbyte;
@@ -402,32 +449,36 @@ get_tag( Sockbuf *sb )
        }
 
        /* tag too big! */
-       if ( i == sizeof(long) )
+       if ( i == sizeof(ber_tag_t) )
                return( LBER_DEFAULT );
 
        /* want leading, not trailing 0's */
-       return( tag >> (sizeof(long) - i - 1) );
+       return( tag >> (sizeof(ber_tag_t) - i - 1) );
 }
 #endif
 
 /*
  * A rewrite of ber_get_next that can safely be called multiple times 
- * for the same packet. It will simply continue were it stopped until
+ * for the same packet. It will simply continue where it stopped until
  * a full packet is read.
  */
 
-unsigned long
-ber_get_next( Sockbuf *sb, unsigned long *len, BerElement *ber )
+ber_tag_t
+ber_get_next(
+       Sockbuf *sb,
+       ber_len_t *len,
+       BerElement *ber )
 {
        assert( sb != NULL );
        assert( len != NULL );
        assert( ber != NULL );
 
-       if ( ber->ber_debug ) {
-               ber_log_printf( LDAP_DEBUG_TRACE, ber->ber_debug,
-                       "ber_get_next\n" );
-       }
-       
+       assert( SOCKBUF_VALID( sb ) );
+       assert( BER_VALID( ber ) );
+
+       ber_log_printf( LDAP_DEBUG_TRACE, ber->ber_debug,
+               "ber_get_next\n" );
+
        /*
         * Any ber element looks like this: tag length contents.
         * Assuming everything's ok, we return the tag byte (we
@@ -441,17 +492,21 @@ ber_get_next( Sockbuf *sb, unsigned long *len, BerElement *ber )
         */
        
        if (ber->ber_rwptr == NULL) {
-               assert( ber->ber_buf == NULL );
+               /* XXYYZ
+                * dtest does like this assert.
+                */
+               /* assert( ber->ber_buf == NULL ); */
                ber->ber_rwptr = (char *) &ber->ber_tag;
                ber->ber_tag = 0;
        }
 
-#define PTR_IN_VAR( ptr, var )\
-(((ptr)>=(char *) &(var)) && ((ptr)< (char *) &(var)+sizeof(var)))
+#undef PTR_IN_VAR
+#define PTR_IN_VAR( ptr, var ) \
+       (((ptr)>=(char *) &(var)) && ((ptr)< (char *) &(var)+sizeof(var)))
        
        if (PTR_IN_VAR(ber->ber_rwptr, ber->ber_tag)) {
                if (ber->ber_rwptr == (char *) &ber->ber_tag) {
-                       if (ber_pvt_sb_read( sb, ber->ber_rwptr, 1)<=0)
+                       if (ber_int_sb_read( sb, ber->ber_rwptr, 1)<=0)
                                return LBER_DEFAULT;
                        if ((ber->ber_rwptr[0] & LBER_BIG_TAG_MASK)
                                != LBER_BIG_TAG_MASK) {
@@ -463,26 +518,30 @@ ber_get_next( Sockbuf *sb, unsigned long *len, BerElement *ber )
                }
                do {
                        /* reading the tag... */
-                       if (ber_pvt_sb_read( sb, ber->ber_rwptr, 1)<=0)
+                       if (ber_int_sb_read( sb, ber->ber_rwptr, 1)<=0) {
                                return LBER_DEFAULT;
+                       }
+
                        if (! (ber->ber_rwptr[0] & LBER_MORE_TAG_MASK) ) {
                                ber->ber_tag>>=sizeof(ber->ber_tag) -
                                  ((char *) &ber->ber_tag - ber->ber_rwptr);
                                ber->ber_rwptr = (char *) &ber->ber_usertag;
                                goto get_lenbyte;
                        }
-               } while (PTR_IN_VAR(ber->ber_rwptr,ber->ber_tag));
+               } while( PTR_IN_VAR(ber->ber_rwptr, ber->ber_tag ));
+
                errno = ERANGE; /* this is a serious error. */
                return LBER_DEFAULT;
        }
+
 get_lenbyte:
        if (ber->ber_rwptr==(char *) &ber->ber_usertag) {
                unsigned char c;
-               if (ber_pvt_sb_read( sb, (char *) &c, 1)<=0)
+               if (ber_int_sb_read( sb, (char *) &c, 1)<=0)
                        return LBER_DEFAULT;
-               if (c & 0x80) {
-                       int len = c & 0x7f;
-                       if ( (len==0) || ((unsigned) len>sizeof( ber->ber_len ) ) ) {
+               if (c & 0x80U) {
+                       int len = c & 0x7fU;
+                       if ( (len==0) || ( len>sizeof( ber->ber_len ) ) ) {
                                errno = ERANGE;
                                return LBER_DEFAULT;
                        }
@@ -494,29 +553,29 @@ get_lenbyte:
                        goto fill_buffer;
                }
        }
+
        if (PTR_IN_VAR(ber->ber_rwptr, ber->ber_len)) {
-               int res;
-               int to_go;
+               unsigned char netlen[sizeof(ber_len_t)];
+
+               ber_slen_t res;
+               ber_slen_t to_go;
                to_go = (char *) &ber->ber_len + sizeof( ber->ber_len ) -
                        ber->ber_rwptr;
                assert( to_go > 0 );
-               res = ber_pvt_sb_read( sb, ber->ber_rwptr, to_go );
-               if (res <=0)
+               res = BerRead( sb, netlen, to_go );
+               if (res <= 0) {
                        return LBER_DEFAULT;
+               }
                ber->ber_rwptr += res;
-               if (res==to_go) {
-                       /* convert length. */
-                       ber->ber_len = AC_NTOHL( ber->ber_len );
-                       goto fill_buffer;
-               } else {
-#if defined( EWOULDBLOCK )
-                       errno = EWOULDBLOCK;
-#elif defined( EAGAIN )
-                       errno = EAGAIN;
-#endif                 
-                       return LBER_DEFAULT;
+
+               /* convert length. */
+               ber->ber_len = 0;
+               for( to_go = 0; to_go < res ; to_go++ ) {
+                       ber->ber_len <<= 8;
+                       ber->ber_len |= netlen[to_go];
                }
        }
+
 fill_buffer:   
        /* now fill the buffer. */
        if (ber->ber_buf==NULL) {
@@ -524,21 +583,22 @@ fill_buffer:
                        errno = ERANGE;
                        return LBER_DEFAULT;
                }
-               ber->ber_buf = (char *) malloc( ber->ber_len );
+               ber->ber_buf = (char *) LBER_MALLOC( ber->ber_len );
                if (ber->ber_buf==NULL)
                        return LBER_DEFAULT;
                ber->ber_rwptr = ber->ber_buf;
                ber->ber_ptr = ber->ber_buf;
                ber->ber_end = ber->ber_buf + ber->ber_len;
        }
+
        if ((ber->ber_rwptr>=ber->ber_buf) && (ber->ber_rwptr<ber->ber_end)) {
-               int res;
-               int to_go;
+               ber_slen_t res;
+               ber_slen_t to_go;
                
                to_go = ber->ber_end - ber->ber_rwptr;
                assert( to_go > 0 );
                
-               res = ber_pvt_sb_read( sb, ber->ber_rwptr, to_go );
+               res = ber_int_sb_read( sb, ber->ber_rwptr, to_go );
                if (res<=0)
                        return LBER_DEFAULT;
                ber->ber_rwptr+=res;
@@ -565,15 +625,3 @@ fill_buffer:
        assert( 0 ); /* ber structure is messed up ?*/
        return LBER_DEFAULT;
 }
-
-void   ber_clear( BerElement *ber, int freebuf )
-{
-       assert( ber != NULL );
-
-       if ((freebuf) && (ber->ber_buf))
-               free( ber->ber_buf );
-       ber->ber_buf = NULL;
-       ber->ber_rwptr = NULL;
-       ber->ber_end = NULL;
-}
-