access past array bounds in layerInterp_mdisps, also make some vars const.
authorCampbell Barton <ideasman42@gmail.com>
Sat, 12 Feb 2011 10:18:21 +0000 (10:18 +0000)
committerCampbell Barton <ideasman42@gmail.com>
Sat, 12 Feb 2011 10:18:21 +0000 (10:18 +0000)
source/blender/blenkernel/BKE_multires.h
source/blender/blenkernel/intern/customdata.c
source/blender/blenkernel/intern/multires.c

index 23fabce..de895eb 100644 (file)
@@ -84,12 +84,12 @@ void multires_mdisp_smooth_bounds(struct MDisps *disps);
 void multires_topology_changed(struct Scene *scene, struct Object *ob);
 
 /**** interpolation stuff ****/
-void old_mdisps_bilinear(float out[3], float (*disps)[3], int st, float u, float v);
-void mdisp_rot_crn_to_face(int S, int corners, int face_side, float x, float y, float *u, float *v);
-int mdisp_rot_face_to_crn(int corners, int face_side, float u, float v, float *x, float *y);
-void mdisp_apply_weight(int S, int corners, int x, int y, int face_side, float crn_weight[4][2], float *u_r, float *v_r);
-void mdisp_flip_disp(int S, int corners, float axis_x[2], float axis_y[2], float disp[3]);
+void old_mdisps_bilinear(float out[3], float (*disps)[3], const int st, float u, float v);
+void mdisp_rot_crn_to_face(const int S, const int corners, const int face_side, const float x, const float y, float *u, float *v);
+int mdisp_rot_face_to_crn(const int corners, const int face_side, const float u, const float v, float *x, float *y);
+void mdisp_apply_weight(const int S, const int corners, int x, int y, const int face_side, float crn_weight[4][2], float *u_r, float *v_r);
+void mdisp_flip_disp(const int S, const int corners, const float axis_x[2], const float axis_y[2], float disp[3]);
 void mdisp_join_tris(struct MDisps *dst, struct MDisps *tri1, struct MDisps *tri2);
 
-#endif
+#endif // BKE_MULTIRES_H
 
index 94bb771..0ed479b 100644 (file)
@@ -524,8 +524,8 @@ static void layerInterp_mdisps(void **sources, float *UNUSED(weights),
                mdisp_apply_weight(S, dst_corners, side-1, 0, st, crn_weight, &axis_x[0], &axis_x[1]);
                mdisp_apply_weight(S, dst_corners, 0, side-1, st, crn_weight, &axis_y[0], &axis_y[1]);
 
-               sub_v3_v3(axis_x, base);
-               sub_v3_v3(axis_y, base);
+               sub_v2_v2(axis_x, base);
+               sub_v2_v2(axis_y, base);
                normalize_v2(axis_x);
                normalize_v2(axis_y);
 
index 2a3052d..2601adb 100644 (file)
@@ -962,7 +962,7 @@ DerivedMesh *multires_dm_create_from_derived(MultiresModifierData *mmd, int loca
 ***************************/
 
 /* Adapted from sculptmode.c */
-void old_mdisps_bilinear(float out[3], float (*disps)[3], int st, float u, float v)
+void old_mdisps_bilinear(float out[3], float (*disps)[3], const int st, float u, float v)
 {
        int x, y, x2, y2;
        const int st_max = st - 1;
@@ -1965,7 +1965,7 @@ static void face_to_crn_interp(float u, float v, float v1[2], float v2[2], float
        *x = maxf(x1, x2);
 }
 
-void mdisp_rot_crn_to_face(int S, int corners, int face_side, float x, float y, float *u, float *v)
+void mdisp_rot_crn_to_face(const int S, const int corners, const int face_side, const float x, const float y, float *u, float *v)
 {
        float offset = face_side*0.5f - 0.5f;
 
@@ -1992,9 +1992,9 @@ void mdisp_rot_crn_to_face(int S, int corners, int face_side, float x, float y,
        }
 }
 
-int mdisp_rot_face_to_crn(int corners, int face_side, float u, float v, float *x, float *y)
+int mdisp_rot_face_to_crn(const int corners, const int face_side, const float u, const float v, float *x, float *y)
 {
-       float offset = face_side*0.5f - 0.5f;
+       const float offset = face_side*0.5f - 0.5f;
        int S = 0;
 
        if (corners == 4) {
@@ -2037,7 +2037,7 @@ int mdisp_rot_face_to_crn(int corners, int face_side, float u, float v, float *x
        return S;
 }
 
-void mdisp_apply_weight(int S, int corners, int x, int y, int face_side,
+void mdisp_apply_weight(const int S, const int corners, int x, int y, const int face_side,
        float crn_weight[4][2], float *u_r, float *v_r)
 {
        float u, v, xl, yl;
@@ -2071,7 +2071,7 @@ void mdisp_apply_weight(int S, int corners, int x, int y, int face_side,
        *v_r = mid3[1];
 }
 
-void mdisp_flip_disp(int S, int corners, float axis_x[2], float axis_y[2], float disp[3])
+void mdisp_flip_disp(const int S, const int corners, const float axis_x[2], const float axis_y[2], float disp[3])
 {
        float crn_x[2], crn_y[2];
        float vx[2], vy[2], coord[2];