isl_convex_hull.c: extend: fix error handling

[platform/upstream/isl.git] / isl_convex_hull.c

diff --git a/isl_convex_hull.c b/isl_convex_hull.c
index 362c378..f783671 100644 (file)
--- a/isl_convex_hull.c
+++ b/isl_convex_hull.c
@@ -381,6 +381,7 @@ isl_int *isl_set_wrap_facet(__isl_keep isl_set *set,
        isl_int *facet, isl_int *ridge)
 {
        int i;
+       isl_ctx *ctx;
        struct isl_mat *T = NULL;
        struct isl_basic_set *lp = NULL;
        struct isl_vec *obj;
@@ -388,11 +389,14 @@ isl_int *isl_set_wrap_facet(__isl_keep isl_set *set,
        isl_int num, den;
        unsigned dim;
 
+       if (!set)
+               return NULL;
+       ctx = set->ctx;
        set = isl_set_copy(set);
        set = isl_set_set_rational(set);
 
        dim = 1 + isl_set_n_dim(set);
-       T = isl_mat_alloc(set->ctx, 3, dim);
+       T = isl_mat_alloc(ctx, 3, dim);
        if (!T)
                goto error;
        isl_int_set_si(T->row[0][0], 1);
@@ -405,7 +409,7 @@ isl_int *isl_set_wrap_facet(__isl_keep isl_set *set,
        if (!set)
                goto error;
        lp = wrap_constraints(set);
-       obj = isl_vec_alloc(set->ctx, 1 + dim*set->n);
+       obj = isl_vec_alloc(ctx, 1 + dim*set->n);
        if (!obj)
                goto error;
        isl_int_set_si(obj->block.data[0], 0);
@@ -417,7 +421,7 @@ isl_int *isl_set_wrap_facet(__isl_keep isl_set *set,
        isl_int_init(num);
        isl_int_init(den);
        res = isl_basic_set_solve_lp(lp, 0,
-                           obj->block.data, set->ctx->one, &num, &den, NULL);
+                           obj->block.data, ctx->one, &num, &den, NULL);
        if (res == isl_lp_ok) {
                isl_int_neg(num, num);
                isl_seq_combine(facet, num, facet, den, ridge, dim);
@@ -427,7 +431,9 @@ isl_int *isl_set_wrap_facet(__isl_keep isl_set *set,
        isl_vec_free(obj);
        isl_basic_set_free(lp);
        isl_set_free(set);
-       isl_assert(set->ctx, res == isl_lp_ok || res == isl_lp_unbounded, 
+       if (res == isl_lp_error)
+               return NULL;
+       isl_assert(ctx, res == isl_lp_ok || res == isl_lp_unbounded, 
                   return NULL);
        return facet;
 error:
@@ -617,11 +623,13 @@ static struct isl_basic_set *extend(struct isl_basic_set *hull,
                hull_facet = isl_basic_set_add_equality(hull_facet, hull->ineq[i]);
                hull_facet = isl_basic_set_gauss(hull_facet, NULL);
                hull_facet = isl_basic_set_normalize_constraints(hull_facet);
-               if (!facet)
+               if (!facet || !hull_facet)
                        goto error;
                hull = isl_basic_set_cow(hull);
                hull = isl_basic_set_extend_dim(hull,
                        isl_dim_copy(hull->dim), 0, 0, facet->n_ineq);
+               if (!hull)
+                       goto error;
                for (j = 0; j < facet->n_ineq; ++j) {
                        for (f = 0; f < hull_facet->n_ineq; ++f)
                                if (isl_seq_eq(facet->ineq[j],
@@ -1802,6 +1810,9 @@ static struct isl_basic_set *uset_convex_hull_wrap_bounded(struct isl_set *set)
 {
        struct isl_basic_set *convex_hull = NULL;
 
+       if (!set)
+               goto error;
+
        if (isl_set_n_dim(set) == 0) {
                convex_hull = isl_basic_set_universe(isl_dim_copy(set->dim));
                isl_set_free(set);
@@ -1810,9 +1821,6 @@ static struct isl_basic_set *uset_convex_hull_wrap_bounded(struct isl_set *set)
        }
 
        set = isl_set_set_rational(set);
-
-       if (!set)
-               goto error;
        set = isl_set_coalesce(set);
        if (!set)
                goto error;