This is required in apptype.cc also:

diff --git a/osaf/services/saf/amf/amfd/apptype.cc 
b/osaf/services/saf/amf/amfd/apptype.cc
--- a/osaf/services/saf/amf/amfd/apptype.cc
+++ b/osaf/services/saf/amf/amfd/apptype.cc
@@ -110,6 +110,7 @@ static AVD_APP_TYPE *apptype_create(SaNa

         app_type = new AVD_APP_TYPE();

+       memset(&app_type->name, 0, sizeof(SaNameT));
         memcpy(app_type->name.value, dn->value, dn->length);
         app_type->name.length = dn->length;



On 04-Aug-14 10:50 AM, Gary Lee wrote:
>   osaf/services/saf/amf/amfd/app.cc |  1 +
>   1 files changed, 1 insertions(+), 0 deletions(-)
>
>
> Ensure app.name.value is null terminated. It seems some code
> assumes SaNameT.value is null terminated, eg. ccbutil_getCcbOpDataByDN()
> in immutil.c.
>
> diff --git a/osaf/services/saf/amf/amfd/app.cc 
> b/osaf/services/saf/amf/amfd/app.cc
> --- a/osaf/services/saf/amf/amfd/app.cc
> +++ b/osaf/services/saf/amf/amfd/app.cc
> @@ -45,6 +45,7 @@ AVD_APP::AVD_APP(const SaNameT* dn) :
>       app_type_list_app_next(NULL),
>       app_type(NULL)
>   {
> +     memset(&name, 0, sizeof(SaNameT));
>       memcpy(name.value, dn->value, dn->length);
>       name.length = dn->length;
>       memset(&saAmfAppType, 0, sizeof(SaNameT));
>

------------------------------------------------------------------------------
Infragistics Professional
Build stunning WinForms apps today!
Reboot your WinForms applications with our WinForms controls. 
Build a bridge from your legacy apps to the future.
http://pubads.g.doubleclick.net/gampad/clk?id=153845071&iu=/4140/ostg.clktrk
_______________________________________________
Opensaf-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/opensaf-devel

Reply via email to