<br><br><div class="gmail_quote">On Thu Jan 15 2015 at 12:45:58 PM <<a href="mailto:beandog@gentoo.org">beandog@gentoo.org</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">From: Steve Dibb <<a href="mailto:steve.dibb@gmail.com" target="_blank">steve.dibb@gmail.com</a>><br>
<br>
---<br>
 src/dvdread/ifo_read.h |  8 +++++++-<br>
 src/ifo_read.c         | 12 ++++++------<br>
 2 files changed, 13 insertions(+), 7 deletions(-)<br>
<br>
diff --git a/src/dvdread/ifo_read.h b/src/dvdread/ifo_read.h<br>
index 97f4179..e9e5568 100644<br>
--- a/src/dvdread/ifo_read.h<br>
+++ b/src/dvdread/ifo_read.h<br>
@@ -219,10 +219,16 @@ void ifoFree_PGCI_UT(ifo_handle_t *);<br>
 void ifoFree_VTS_TMAPT(ifo_handle_t *);<br>
 void ifoFree_C_ADT(ifo_handle_t *);<br>
 void ifoFree_TITLE_C_ADT(ifo_<u></u>handle_t *);<br>
-void ifoFree_VOBU_ADMAP(ifo_handle_<u></u>t *);<br>
 void ifoFree_TITLE_VOBU_ADMAP(ifo_<u></u>handle_t *);<br>
 void ifoFree_TXTDT_MGI(ifo_handle_t *);<br>
<br>
+/**<br>
+ * Deprecated functions used internally for freeing parsed sections of<br>
+ * the ifo_handle_t structure and the allocated substructures.  These<br>
+ * perform no function -- use ifoFree() only.<br>
+ */<br>
</blockquote><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">+void ifoFree_VOBU_ADMAP(ifo_handle_<u></u>t *);<br></blockquote><div><br></div><div><div style="font-size:13.3333330154419px">#if __GNUC__ > 3 || __GNUC__ == 3 && __GNUC_MINOR__ >= 1</div><div style="font-size:13.3333330154419px"># define attribute_deprecated __attribute__((deprecated))<br></div><div style="font-size:13.3333330154419px">#else</div><div style="font-size:13.3333330154419px"># define attribute_deprecated</div><div style="font-size:13.3333330154419px"><span style="font-size:13.3333330154419px">#endif</span></div><div style="font-size:13.3333330154419px"><span style="font-size:13.3333330154419px">attribute_deprecated </span><span style="font-size:13.3333330154419px">void ifoFree_VOBU_ADMAP(ifo_handle_</span><u style="font-size:13.3333330154419px"></u><span style="font-size:13.3333330154419px">t *);</span></div></div><div><br></div><div>The deprecated attribute will print a compiler warning on all client apps using the function.</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
+<br>
 #ifdef __cplusplus<br>
 };<br>
 #endif<br>
diff --git a/src/ifo_read.c b/src/ifo_read.c<br>
index ec6952f..1310911 100644<br>
--- a/src/ifo_read.c<br>
+++ b/src/ifo_read.c<br>
@@ -487,7 +487,11 @@ void ifoClose(ifo_handle_t *ifofile) {<br>
   if(!ifofile)<br>
     return;<br>
<br>
-  ifoFree_VOBU_ADMAP(ifofile);<br>
+  if(ifofile->menu_vobu_admap) {<br>
+    free(ifofile->menu_vobu_admap-<u></u>>vobu_start_sectors);<br>
+    free(ifofile->menu_vobu_admap)<u></u>;<br>
+  }<br>
+<br>
   ifoFree_TITLE_VOBU_ADMAP(<u></u>ifofile);<br>
   ifoFree_C_ADT(ifofile);<br>
   ifoFree_TITLE_C_ADT(ifofile);<br>
@@ -1829,11 +1833,7 @@ static void ifoFree_VOBU_ADMAP_internal(<u></u>vobu_admap_t *vobu_admap) {<br>
 }<br>
<br>
 void ifoFree_VOBU_ADMAP(ifo_handle_<u></u>t *ifofile) {<br>
-  if(!ifofile)<br>
-    return;<br>
-<br>
-  ifoFree_VOBU_ADMAP_internal(<u></u>ifofile->menu_vobu_admap);<br>
-  ifofile->menu_vobu_admap = NULL;<br>
+  return;<br></blockquote><div><br></div><div>This return shouldn't be needed.</div><div><br></div><div>Timothy</div></div>