r16542: Fix #3863, reported by jason@ncac.gwu.edu
authorJeremy Allison <jra@samba.org>
Tue, 27 Jun 2006 00:42:24 +0000 (00:42 +0000)
committerGerald (Jerry) Carter <jerry@samba.org>
Wed, 10 Oct 2007 16:18:59 +0000 (11:18 -0500)
Jeremy.

source/rpc_server/srv_eventlog_lib.c
source/rpc_server/srv_eventlog_nt.c

index e04aefa7e1b3a6abd00abd7f3f21ca50238189d0..66be1dc34ff58e74ddfc5de710cb1c773f53858f 100644 (file)
@@ -201,6 +201,12 @@ BOOL make_way_for_eventlogs( TDB_CONTEXT * the_tdb, int32 needed,
 
                len = tdb_unpack( ret.dptr, ret.dsize, "ddddd", &reclen,
                                  &tresv1, &trecnum, &timegen, &timewr );
+               if (len == -1) {
+                       DEBUG( 10,("make_way_for_eventlogs: tdb_unpack failed.\n"));
+                       tdb_unlock_bystring( the_tdb, EVT_NEXT_RECORD );
+                       return False;
+               }
+
                DEBUG( 8,
                       ( "read record %d, record size is [%d], total so far [%d]\n",
                         i, reclen, nbytes ) );
index c1c0b6a0e24317b3455d03be63a9ced5f2206c41..79839a0a52a67e9b01c65c66a928915264a0a89c 100644 (file)
@@ -289,12 +289,10 @@ Eventlog_entry *get_eventlog_record( prs_struct * ps, TDB_CONTEXT * tdb,
        int srecno;
        int reclen;
        int len;
-       uint8 *rbuff;
 
        pstring *wpsource, *wpcomputer, *wpsid, *wpstrs, *puserdata;
 
        key.dsize = sizeof( int32 );
-       rbuff = NULL;
 
        srecno = recno;
        key.dptr = ( char * ) &srecno;