[SCM] Samba Shared Repository - branch master updated

[Jeremy Allison]

The branch, master has been updated
       via  df13b13... Fix bug #7154 - mangling method = hash can crash storing a name not containing a '.'
      from  0bf9c5e... testprogs: add RPC_S_PROCNUM_OUT_OF_RANGE error.

http://gitweb.samba.org/?p=samba.git;a=shortlog;h=master


- Log -----------------------------------------------------------------
commit df13b1303a751962d8f7d5298b39e4a7500fef15
Author: Jeremy Allison <jra at samba.org>
Date:   Thu Feb 18 11:22:44 2010 -0800

    Fix bug #7154 - mangling method = hash can crash storing a name not containing a '.'
    
    Fix use of uninitialized variable. This can lead to crashes if
    mangling = hash processes names with no '.'.
    
    Jeremy.

-----------------------------------------------------------------------

Summary of changes:
 source3/smbd/mangle_hash.c |    8 +++++---
 1 files changed, 5 insertions(+), 3 deletions(-)


Changeset truncated at 500 lines:

diff --git a/source3/smbd/mangle_hash.c b/source3/smbd/mangle_hash.c
index 8369af4..1482d10 100644
--- a/source3/smbd/mangle_hash.c
+++ b/source3/smbd/mangle_hash.c
@@ -408,8 +408,8 @@ static void cache_mangled_name( const char mangled_name[13],
 {
 	TDB_DATA data_val;
 	char mangled_name_key[13];
-	char *s1;
-	char *s2;
+	char *s1 = NULL;
+	char *s2 = NULL;
 
 	/* If the cache isn't initialized, give up. */
 	if( !tdb_mangled_cache )
@@ -448,7 +448,9 @@ static void cache_mangled_name( const char mangled_name[13],
 		DEBUG(5,("cache_mangled_name: Stored entry %s -> %s\n", mangled_name_key, raw_name));
 	}
 	/* Restore the change we made to the const string. */
-	*s2 = '.';
+	if (s2) {
+		*s2 = '.';
+	}
 }
 
 /* ************************************************************************** **


-- 
Samba Shared Repository