1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
|
From f031d8492f0acfc0f38e8e5e4e01ed87a22234b2 Mon Sep 17 00:00:00 2001
From: Remi Collet <fedora@famillecollet.com>
Date: Mon, 13 Mar 2017 06:57:14 +0100
Subject: [PATCH] adapt buffer size and secure its usage, fix #273
---
amqp_timestamp.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/amqp_timestamp.c b/amqp_timestamp.c
index 70013f1..79b0e52 100644
--- a/amqp_timestamp.c
+++ b/amqp_timestamp.c
@@ -119,7 +119,7 @@ zend_function_entry amqp_timestamp_class_functions[] = {
PHP_MINIT_FUNCTION(amqp_timestamp)
{
zend_class_entry ce;
- char min[1], max[20];
+ char min[20], max[20];
int min_len, max_len;
INIT_CLASS_ENTRY(ce, "AMQPTimestamp", amqp_timestamp_class_functions);
@@ -128,10 +128,10 @@ PHP_MINIT_FUNCTION(amqp_timestamp)
zend_declare_property_null(this_ce, ZEND_STRL("timestamp"), ZEND_ACC_PRIVATE TSRMLS_CC);
- max_len = sprintf(max, "%.0f", AMQP_TIMESTAMP_MAX);
+ max_len = snprintf(max, sizeof(max), "%.0f", AMQP_TIMESTAMP_MAX);
zend_declare_class_constant_stringl(this_ce, ZEND_STRL("MAX"), max, max_len TSRMLS_CC);
- min_len = sprintf(min, "%.0f", AMQP_TIMESTAMP_MIN);
+ min_len = snprintf(min, sizeof(min), "%.0f", AMQP_TIMESTAMP_MIN);
zend_declare_class_constant_stringl(this_ce, ZEND_STRL("MIN"), min, min_len TSRMLS_CC);
return SUCCESS;
|