|
 |
304803 |
autofs-5.1.0-beta1 - fix incorrect max key length in defaults get_hash()
|
|
|
304803 |
|
|
|
304803 |
From: Ian Kent <ikent@redhat.com>
|
|
|
304803 |
|
|
|
304803 |
amd section names can correspond to mount point path names.
|
|
|
304803 |
---
|
|
|
304803 |
CHANGELOG | 1 +
|
|
|
304803 |
lib/defaults.c | 6 ++++--
|
|
|
304803 |
2 files changed, 5 insertions(+), 2 deletions(-)
|
|
|
304803 |
|
|
|
304803 |
--- autofs-5.0.7.orig/CHANGELOG
|
|
|
304803 |
+++ autofs-5.0.7/CHANGELOG
|
|
|
304803 |
@@ -114,6 +114,7 @@
|
|
|
304803 |
- fix ldap default schema config.
|
|
|
304803 |
- fix ldap default master map name config.
|
|
|
304803 |
- fix map format init in lookup_init().
|
|
|
304803 |
+- fix incorrect max key length in defaults get_hash().
|
|
|
304803 |
|
|
|
304803 |
25/07/2012 autofs-5.0.7
|
|
|
304803 |
=======================
|
|
|
304803 |
--- autofs-5.0.7.orig/lib/defaults.c
|
|
|
304803 |
+++ autofs-5.0.7/lib/defaults.c
|
|
|
304803 |
@@ -43,7 +43,6 @@
|
|
|
304803 |
#define OLD_CONFIG_FILE AUTOFS_CONF_DIR "/autofs"
|
|
|
304803 |
#define MAX_LINE_LEN 256
|
|
|
304803 |
#define MAX_SECTION_NAME MAX_LINE_LEN
|
|
|
304803 |
-#define MAX_CFG_NAME_LEN 31
|
|
|
304803 |
|
|
|
304803 |
#define NAME_MASTER_MAP "master_map_name"
|
|
|
304803 |
|
|
|
304803 |
@@ -665,7 +664,7 @@ error:
|
|
|
304803 |
static u_int32_t get_hash(const char *key, unsigned int size)
|
|
|
304803 |
{
|
|
|
304803 |
const char *pkey = key;
|
|
|
304803 |
- char lkey[MAX_CFG_NAME_LEN];
|
|
|
304803 |
+ char lkey[PATH_MAX + 1];
|
|
|
304803 |
char *plkey = &lkey[0];
|
|
|
304803 |
|
|
|
304803 |
while (*pkey)
|
|
|
304803 |
@@ -683,6 +682,9 @@ static struct conf_option *conf_lookup(c
|
|
|
304803 |
if (!key || !section)
|
|
|
304803 |
return NULL;
|
|
|
304803 |
|
|
|
304803 |
+ if (strlen(key) > PATH_MAX)
|
|
|
304803 |
+ return NULL;
|
|
|
304803 |
+
|
|
|
304803 |
key_hash = get_hash(key, size);
|
|
|
304803 |
for (co = config->hash[key_hash]; co != NULL; co = co->next) {
|
|
|
304803 |
if (strcasecmp(section, co->section))
|