[libdvdnav-devel] Correctly return fails in dvdnav_jump_to_sector_by_time

Jean-Baptiste Kempf git at videolan.org
Mon Aug 4 19:34:44 CEST 2014


libdvdnav | branch: master | Jean-Baptiste Kempf <jb at videolan.org> | Mon Aug  4 19:30:42 2014 +0200| [42403a82415da3e3728cca2272727d2f357609a4] | committer: Jean-Baptiste Kempf

Correctly return fails in dvdnav_jump_to_sector_by_time

If vm_jump_cell_block failed, we should return fail to
dvdnav_jump_to_sector_by_time() call

> http://git.videolan.org/gitweb.cgi/libdvdnav.git/?a=commit;h=42403a82415da3e3728cca2272727d2f357609a4
---

 src/searching.c |    4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/searching.c b/src/searching.c
index 2421847..81155e3 100644
--- a/src/searching.c
+++ b/src/searching.c
@@ -1287,12 +1287,12 @@ dvdnav_status_t dvdnav_jump_to_sector_by_time(dvdnav_t *this,
 
   /* jump to sector */
   sector_off = jump->sector - cell_data->bgn->sector;
+  result = vm_jump_cell_block(this->vm, cell_data->idx, sector_off);
   this->cur_cell_time = 0;
-  if (vm_jump_cell_block(this->vm, cell_data->idx, sector_off)) {
+  if (result) {
     pthread_mutex_lock(&this->vm_lock);
     this->vm->hop_channel += HOP_SEEK;
     pthread_mutex_unlock(&this->vm_lock);
-    result = DVDNAV_STATUS_OK;
   }
 
 exit:



More information about the libdvdnav-devel mailing list