Mesa (master): st/omx: strcpy the string into the allocated buffer

Emil Velikov evelikov at kemper.freedesktop.org
Sat Jun 28 15:53:22 UTC 2014


Module: Mesa
Branch: master
Commit: 15b5e663b050505683b7b4c9c489e46863b8441d
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=15b5e663b050505683b7b4c9c489e46863b8441d

Author: Emil Velikov <emil.l.velikov at gmail.com>
Date:   Sat Jun 28 15:24:43 2014 +0100

st/omx: strcpy the string into the allocated buffer

This fixes commit a001ca98e15(st/omx: keep the name,
(name|role)_specific strings dynamically allocated) in which we
dynamically allocated the buffers for name and (name|role)_specific
yet forgot to copy the encoder strings into them.

Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=80614
Signed-off-by: Emil Velikov <emil.l.velikov at gmail.com>

---

 src/gallium/state_trackers/omx/vid_enc.c |    6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/gallium/state_trackers/omx/vid_enc.c b/src/gallium/state_trackers/omx/vid_enc.c
index 871b465..83624c5 100644
--- a/src/gallium/state_trackers/omx/vid_enc.c
+++ b/src/gallium/state_trackers/omx/vid_enc.c
@@ -125,9 +125,9 @@ OMX_ERRORTYPE vid_enc_LoaderComponent(stLoaderComponentType *comp)
    if (comp->role_specific[0] == NULL)
       goto error_specific;
 
-   comp->name = OMX_VID_ENC_BASE_NAME;
-   comp->name_specific[0] = OMX_VID_ENC_AVC_NAME;
-   comp->role_specific[0] = OMX_VID_ENC_AVC_ROLE;
+   strcpy(comp->name, OMX_VID_ENC_BASE_NAME);
+   strcpy(comp->name_specific[0], OMX_VID_ENC_AVC_NAME);
+   strcpy(comp->role_specific[0], OMX_VID_ENC_AVC_ROLE);
 
    return OMX_ErrorNone;
 




More information about the mesa-commit mailing list