diff options
author | Vsevolod Stakhov <vsevolod@highsecure.ru> | 2016-06-13 13:10:50 +0100 |
---|---|---|
committer | Vsevolod Stakhov <vsevolod@highsecure.ru> | 2016-06-13 17:31:29 +0100 |
commit | 1ad42d3b00442d7122b16c72d1226f6c2b0e133f (patch) | |
tree | 9a855a4aa0ec54e5d5bec627a6d7464e45352f4b /src/libserver/task.c | |
parent | 121dd2106056508787f8046a89da38becc41c551 (diff) | |
download | rspamd-1ad42d3b00442d7122b16c72d1226f6c2b0e133f.tar.gz rspamd-1ad42d3b00442d7122b16c72d1226f6c2b0e133f.zip |
[Fix] Fix length calculations for shared memory segments
Diffstat (limited to 'src/libserver/task.c')
-rw-r--r-- | src/libserver/task.c | 24 |
1 files changed, 21 insertions, 3 deletions
diff --git a/src/libserver/task.c b/src/libserver/task.c index 82e4d6fa9..f8ebe81f1 100644 --- a/src/libserver/task.c +++ b/src/libserver/task.c @@ -286,7 +286,7 @@ rspamd_task_load_message (struct rspamd_task *task, ucl_object_t *control_obj; gchar filepath[PATH_MAX], *fp; gint fd, flen; - gulong offset = 0; + gulong offset = 0, shmem_size = 0; rspamd_ftok_t srch, *tok; gpointer map; struct stat st; @@ -360,14 +360,32 @@ rspamd_task_load_message (struct rspamd_task *task, } } + srch.begin = "shm-length"; + srch.len = 10; + tok = g_hash_table_lookup (task->request_headers, &srch); + shmem_size = st.st_size; + + if (tok) { + rspamd_strtoul (tok->begin, tok->len, &shmem_size); + + if (shmem_size > (gulong)st.st_size) { + msg_err_task ("invalid length %ul (%ul available) for shm " + "segment %s", shmem_size, st.st_size, fp); + munmap (map, st.st_size); + + return FALSE; + } + } + task->msg.begin = ((guchar *)map) + offset; - task->msg.len = st.st_size - offset; + task->msg.len = shmem_size; task->flags |= RSPAMD_TASK_FLAG_FILE; m = rspamd_mempool_alloc (task->task_pool, sizeof (*m)); m->begin = map; m->len = st.st_size; - msg_info_task ("loaded message from shared memory %s", fp); + msg_info_task ("loaded message from shared memory %s (%ul size, %ul offset)", + fp, shmem_size, offset); rspamd_mempool_add_destructor (task->task_pool, rspamd_task_unmapper, m); |