[Tensor Decoder] Security: fixed not null termination of strncpy(3)
authorGeunsik Lim <geunsik.lim@samsung.com>
Tue, 23 Oct 2018 08:09:54 +0000 (17:09 +0900)
committerjinhyuck-park <35015471+jinhyuck-park@users.noreply.github.com>
Tue, 23 Oct 2018 23:28:44 +0000 (08:28 +0900)
commit631e1d0f0ea2f9ec8a5c9f28ebbe4a02e14cc219
tree23d0725f375fc6d983cd78a14932466205d9c84b
parentc757bda9592798498914f328975d723aba7c56f6
[Tensor Decoder] Security: fixed not null termination of strncpy(3)

Fixed issue #690.

This commit is to fix not null termination issue in strncpy library call
in the tensor decoder element.

strncpy() supposedly protects from buffer overflows. But if it prevents
an overflow without null terminating, in all likelyhood a subsequent
string operation is going to overflow.

**Change proposed in this PR:**
* Version 3:
  1. Commented security, gstreamer structure, and Svace (Won't fix)
  2. Keep Gstreamer (gstbasetextoverlay.c) policy

* Version 2:
  1. Replaced strncpy with memcpy.
  2. Added annotation.

* Version 1:
  1. Fixed not null termination issue of strncpy() by using memset().

Signed-off-by: Geunsik Lim <geunsik.lim@samsung.com>
gst/tensor_decoder/tensordec.c