kernel: Increase +1 size of destination buffer to avoid overflow

Increase the size of the destination buffer to one more than the data to be copied through sscanf.
For example, if we want to copy BUFF_MAX size from source buffer
then destination buffer size should be larger than BUFF_MAX (e.g., BUFF_MAX + 1)
to avoid buffer overflow.
By doing so, regardless of the size of the source buffer,
only up to BUFF_MAX size is copied to the destination buffer.

Change-Id: I63cefd0f325d89fa41b6fe4241f871191c0884fa
Signed-off-by: Unsung Lee <unsung.lee@samsung.com>

diff --git a/src/util/kernel.c b/src/util/kernel.c
index e3dedda80afd536ca96eb14fa95c75303167eab4..c6dd3f8a64c4dbfccb835cd8524d9f3d07a64082 100644 (file)
--- a/src/util/kernel.c
+++ b/src/util/kernel.c
@@ -452,7 +452,7 @@ static inline bool is_new_entry(const char *str)
 static unsigned long get_gpu_mem_size(FILE *smaps_fd, char *buffer)
 {
        unsigned long mem_size = 0;
-       char name[BUFF_MAX];
+       char name[BUFF_MAX + 1];
 
        if (!gpu_mem_node)
                return 0;