s3/smbd: Fix string buffer overflow causing heap corruption
authorSteven Danneman <steven.danneman@isilon.com>
Sat, 30 Jan 2010 21:29:23 +0000 (13:29 -0800)
committerSteven Danneman <steven.danneman@isilon.com>
Sat, 30 Jan 2010 21:38:31 +0000 (13:38 -0800)
commitf42971c520360e69c4cdd64bebb02a5f5ba49b94
treeeb8900699a2eeacb58553a81b87b7b93d7c7861d
parent772d808ac862862330dd792ba8e83ba2b3c785b3
s3/smbd: Fix string buffer overflow causing heap corruption

The destname malloc size was not taking into account the 1 extra byte
needed if a string without a leading '/' was passed in and that slash
was added.

This would cause the '\0' byte to be written past the end of the
malloced destname string and corrupt whatever heap memory was there.

This problem would be hit if a share name was given in smb.conf without
a leading '/' and if it was the exact size of the allocated STRDUP memory
which in some implementations of malloc is a power of 2.
source3/smbd/service.c