[ofa-general] Re: [PATCH] opensm: wrong variable memset()

Hal Rosenstock hal.rosenstock at gmail.com
Sun Aug 5 09:25:01 PDT 2007


Hi Sasha,

On 8/5/07, Sasha Khapyorsky <sashak at voltaire.com> wrote:
>
> Fix wrong variable memset().
>
> Signed-off-by: Sasha Khapyorsky <sashak at voltaire.com>
> ---
>  opensm/opensm/osm_state_mgr.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/opensm/opensm/osm_state_mgr.c b/opensm/opensm/osm_state_mgr.c
> index 1cf6257..2e43f86 100644
> --- a/opensm/opensm/osm_state_mgr.c
> +++ b/opensm/opensm/osm_state_mgr.c
> @@ -587,7 +587,7 @@ __osm_state_mgr_get_sw_info(
>    p_node = p_sw->p_node;
>    p_dr_path = osm_node_get_any_dr_path_ptr( p_node );
>
> -   memset( &context, 0, sizeof( context ) );
> +   memset( &mad_context, 0, sizeof( mad_context ) );

Nice catch. Should this also be applied for 1.2.c/1.2.1 ?

-- Hal

>    mad_context.si_context.node_guid = osm_node_get_node_guid( p_node );
>    mad_context.si_context.set_method = FALSE;
> --
> 1.5.3.rc2.29.gc4640f
>
>



More information about the general mailing list