The other odd thing is this field never ever gets used :-)
I've removed the test.

        Danny

On Fri, 2004-10-01 at 16:31, Dave Williss wrote:
> One of our other engineers found a UMR (Uninitialized Memory Read)
> in Lesstif using Purify. See below...
> 
> A UMR indicates that the line of code in question tried to access part of
> a structure which was allocated with malloc() but was never initialized to
> anything.
> 
> The line in question is...
> 
> -->    if (RCC_WasManaged(kid_geometry->kid))
>              {
>               RCC_WasManaged(kid_geometry->kid) = False;
>              }
> 
> I'm not sure which part of that it thinks is uninitialized.  Although the 
> logic of
> only setting the thing to false if it isn't already seems like an odd sort 
> of thing
> to do.  If it's already False, setting it to False again can't hurt anything 
> and is
> quicker than checking anyway.
> 
> ----- Original Message ----- 
> From: Michael Unverferth
> To: Dave Williss
> Sent: Friday, October 01, 2004 9:11 AM
> Subject: UMR - lesstif
> 
> 
> [W] UMR: Uninitialized memory read in XmRCGetKidGeo {1 occurrence}
>         Reading 1 byte from 0x10d70ae4 (1 byte at 0x10d70ae4 uninitialized)
>         Address 0x10d70ae4 is 316 bytes into a 328 byte block at 0x10d709a8
>         Address 0x10d70ae4 points to a malloc'd block in heap 0x082f0000
>         Thread ID: 0x4ac
>         Error location
>             XmRCGetKidGeo  [rcutils.c:1220]
>             XmRCAdjustSize [rcutils.c:1572]
>             change_managed [rowcolumn.c:1626]
>             CallChangeManaged [intrinsic.c:250]
>             CallChangeManaged [intrinsic.c:242]
>             CallChangeManaged [intrinsic.c:242]
>             XtRealizeWidget [intrinsic.c:403]
>             CreateView     [tvgroup.c:784]
>             TNTviewGroupCreate [tvgroup.c:1529]
>             CB_ActionMain  [tvmain.c:546]
>             XtCallCallbackList [callback.c:660]
>             ButtonUp       [pushbg.c:1663]
>             input_dispatch [pushbg.c:1815]
>             XmDispatchGadgetInput [gadgetutil.c:284]
>             XmMenuBtnUp    [rowcolumn.c:3265]
>             HandleActions  [tmstate.c:630]
>             HandleSimpleState [tmstate.c:872]
>             XtTranslateEvent [tmstate.c:1084]
>             XtDispatchEventToWidget [event.c:976]
>             XtDefaultDispatcher [event.c:1437]
>         Allocation location
>             malloc         [dbgheap.c:138]
>             XtMalloc       [alloc.c:120]
>             _XtMalloc      [alloc.c:172]
>             xtWidgetAlloc  [create.c:289]
>             xtCreate       [create.c:350]
>             XtCreateWidget [create.c:557]
>             XtCreateWidget [create.c:585]
>             XmCreateCascadeButton [cascadeb.c:1364]
>             BuildMenu      [bldmenu.c:641]
>             MxCreateMenuBar [bldmenu.c:769]
>             GRE_VIEW::CreateMenuBar(_WidgetRec *,MENUBARITEM *,UINT) 
> [mdview.c:7723]
>             GRE_VIEW::CreateMenuBar(_WidgetRec *,UINT) [mdview.c:7773]
>             CreateView     [tvgroup.c:762]
>             TNTviewGroupCreate [tvgroup.c:1529]
>             CB_ActionMain  [tvmain.c:546] 
> 
> 
> 
> -------------------------------------------------------
> This SF.net email is sponsored by: IT Product Guide on ITManagersJournal
> Use IT products in your business? Tell us what you think of them. Give us
> Your Opinions, Get Free ThinkGeek Gift Certificates! Click to find out more
> http://productguide.itmanagersjournal.com/guidepromo.tmpl
> _______________________________________________
> Lesstif-discuss mailing list
> [EMAIL PROTECTED]
> https://lists.sourceforge.net/lists/listinfo/lesstif-discuss
-- 
Danny Backx - danny.backx-at-planetinternet.be    http://up.to/danny.backx



-------------------------------------------------------
This SF.net email is sponsored by: IT Product Guide on ITManagersJournal
Use IT products in your business? Tell us what you think of them. Give us
Your Opinions, Get Free ThinkGeek Gift Certificates! Click to find out more
http://productguide.itmanagersjournal.com/guidepromo.tmpl
_______________________________________________
Lesstif-discuss mailing list
[EMAIL PROTECTED]
https://lists.sourceforge.net/lists/listinfo/lesstif-discuss

Reply via email to