Some possible issues with liblrdf

[Leland (AudacityTeam)]

Hi Lars,

Just fiddling around with trying to integrate liblrdf on OSX and
Windows.  Ran across a couple of things that you may want to look into.  
I didn't go real deep, so there may be more than these.

The following functions will leak memory if no (sub|super)classes are found:

    lrdf_get_all_subclasses()
    lrdf_get_all_superclasses()

Because the above functions can return NULL, lrdf_get_all_instances()
runs risk of using NULL ptr.

Because the following functions do not use lrdf_uris_new(), the
lrdf_uris they return will not be usable by lrdf_uris_append():

    lrdf_get_setting_uris()
    lrdf_get_subclasses()

Though not an "error", it'd probably be "cleaner" if lrdf_uris allocated
via lrdf_uris_new() would be freed via lrdf_uris_free().

Leland


   

-------------------------------------------------------------------------
Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
Studies have shown that voting for your favorite open source project,
along with a healthy diet, reduces your potential for chronic lameness
and boredom. Vote Now at http://www.sourceforge.net/community/cca08
_______________________________________________
Audacity-devel mailing list
Audacity-devel@...
https://lists.sourceforge.net/lists/listinfo/audacity-devel

[Lars Luthman-3]

On Thu, 2008-07-03 at 11:06 -0500, Leland wrote:
> Hi Lars,
>
> Just fiddling around with trying to integrate liblrdf on OSX and
> Windows.  Ran across a couple of things that you may want to look into.  
> I didn't go real deep, so there may be more than these.

Thanks, I'll have a look.


--ll


-------------------------------------------------------------------------
Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
Studies have shown that voting for your favorite open source project,
along with a healthy diet, reduces your potential for chronic lameness
and boredom. Vote Now at http://www.sourceforge.net/community/cca08
_______________________________________________
Audacity-devel mailing list
Audacity-devel@...
https://lists.sourceforge.net/lists/listinfo/audacity-devel