depau/libfprint
1
0
Fork 0
Code Issues Pull requests Projects Releases Wiki Activity

mindtct: Fix memory leaks in get_centered_contour()

Browse source 
The first contour wasn't freed if we failed on the second contour.

libfprint/nbis/mindtct/contour.c:521:14: warning: Potential leak of memory pointed to by 'half1_ex'
      return(ret);
             ^~~
libfprint/nbis/mindtct/contour.c:521:14: warning: Potential leak of memory pointed to by 'half1_ey'
      return(ret);
             ^~~
libfprint/nbis/mindtct/contour.c:521:14: warning: Potential leak of memory pointed to by 'half1_x'
      return(ret);
             ^~~
libfprint/nbis/mindtct/contour.c:521:14: warning: Potential leak of memory pointed to by 'half1_y'
      return(ret);
             ^~~
This commit is contained in:
Bastien Nocera 2018-12-12 17:14:21 +01:00
parent 74bb899ce2
commit eaa4aa964c
1 changed files with 2 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
libfprint/nbis/mindtct/contour.c
View file

@ -517,6 +517,8 @@ int get_centered_contour(int **ocontour_x, int **ocontour_y,
/* If system error occurred on 2nd trace ... */ /* If system error occurred on 2nd trace ... */
if(ret < 0){ if(ret < 0){
/* Deallocate loop's contour. */
free_contour(half1_x, half1_y, half1_ex, half1_ey);
/* Return error code. */ /* Return error code. */
return(ret); return(ret);
} }