Просмотр исходного кода

cpg: Fix signedness of munmap result

munmap is signed and return value of zcb_free is also signed so storing
munmap result into unsigned value doesn't make sense.

Similar issue is fixed in zcb_by_addr_free function.

Signed-off-by: Jan Friesse <jfriesse@redhat.com>
Reviewed-by: Christine Caulfield <ccaulfie@redhat.com>
Jan Friesse 6 месяцев назад
Родитель
Сommit
18c2d235cb
1 измененных файлов с 2 добавлено и 2 удалено
  1. 2 2
      exec/cpg.c

+ 2 - 2
exec/cpg.c

@@ -1770,7 +1770,7 @@ static inline int zcb_alloc (
 
 static inline int zcb_free (struct zcb_mapped *zcb_mapped)
 {
-	unsigned int res;
+	int res;
 
 	res = munmap (zcb_mapped->addr, zcb_mapped->size);
 	qb_list_del (&zcb_mapped->list);
@@ -1782,7 +1782,7 @@ static inline int zcb_by_addr_free (struct cpg_pd *cpd, void *addr)
 {
 	struct qb_list_head *list, *tmp_iter;
 	struct zcb_mapped *zcb_mapped;
-	unsigned int res = 0;
+	int res = 0;
 
 	qb_list_for_each_safe(list, tmp_iter, &(cpd->zcb_mapped_list_head)) {
 		zcb_mapped = qb_list_entry (list, struct zcb_mapped, list);