Keywords: fix bug 135
This changeset fixes a potential stack bashing in the mapfam permutation
code by inserting proper break statements where they belong. This could
be one of the reasons of bug #135 and it's yet to be shown that it fixes
anything (other than pleasing coverity).
* src/map.c (__perm_nseq): no switch fallthrough, insert breaks
Signed-off-by: Sebastian Freundt <hroptatyr@fresse.org>
__advance_multi_index_2(idx, 2, nseqsz);
}
}
+ break;
case 3:
if (LIKELY(!NILP(fun) && gf == NULL)) {
__advance_multi_index_2(idx, 3, nseqsz);
}
}
-
+ break;
default:
if (LIKELY(!NILP(fun) && gf == NULL)) {
__advance_multi_index_2(idx, nseqs, nseqsz);
}
}
+ break;
}
UNGCPRO;