}
#endif
ptr = (prvm_eval_t *)((unsigned char *)prog->edictsfields + OPB->_int);
- ptr->vector[0] = OPA->vector[0];
- ptr->vector[1] = OPA->vector[1];
- ptr->vector[2] = OPA->vector[2];
+ ptr->ivector[0] = OPA->ivector[0];
+ ptr->ivector[1] = OPA->ivector[1];
+ ptr->ivector[2] = OPA->ivector[2];
break;
case OP_ADDRESS:
case OP_LOAD_S:
case OP_LOAD_FNC:
#if PRVMBOUNDSCHECK
+ if (OPA->edict < 0 || OPA->edict >= prog->edictareasize)
+ {
+ prog->xfunction->profile += (st - startst);
+ prog->xstatement = st - prog->statements;
+ PRVM_ERROR ("%s Progs attempted to read an out of bounds edict number", PRVM_NAME);
+ goto cleanup;
+ }
if ((unsigned int)(OPB->_int) >= (unsigned int)(prog->progs->entityfields))
{
prog->xfunction->profile += (st - startst);
case OP_LOAD_V:
#if PRVMBOUNDSCHECK
+ if (OPA->edict < 0 || OPA->edict >= prog->edictareasize)
+ {
+ prog->xfunction->profile += (st - startst);
+ prog->xstatement = st - prog->statements;
+ PRVM_ERROR ("%s Progs attempted to read an out of bounds edict number", PRVM_NAME);
+ goto cleanup;
+ }
if (OPB->_int < 0 || OPB->_int + 2 >= prog->progs->entityfields)
{
prog->xfunction->profile += (st - startst);
}
#endif
ed = PRVM_PROG_TO_EDICT(OPA->edict);
- OPC->vector[0] = ((prvm_eval_t *)((int *)ed->fields.vp + OPB->_int))->vector[0];
- OPC->vector[1] = ((prvm_eval_t *)((int *)ed->fields.vp + OPB->_int))->vector[1];
- OPC->vector[2] = ((prvm_eval_t *)((int *)ed->fields.vp + OPB->_int))->vector[2];
+ OPC->ivector[0] = ((prvm_eval_t *)((int *)ed->fields.vp + OPB->_int))->ivector[0];
+ OPC->ivector[1] = ((prvm_eval_t *)((int *)ed->fields.vp + OPB->_int))->ivector[1];
+ OPC->ivector[2] = ((prvm_eval_t *)((int *)ed->fields.vp + OPB->_int))->ivector[2];
break;
//==================
case OP_IFNOT:
- if (!OPA->_int)
+ if (!OPA->_float)
+ // TODO add an "int-ifnot"
+ // although mostly unneeded, thanks to the only float being false being 0x0 and 0x80000000 (negative zero)
+ // and entity, string, field values can never have that value
{
prog->xfunction->profile += (st - startst);
st += st->b - 1; // offset the s++
startst = st;
+ // no bounds check needed, it is done when loading progs
#if PRVMRUNAWAYCHECK
if (++jumpcount == 10000000)
{
break;
case OP_IF:
- if (OPA->_int)
+ if (OPA->_float)
+ // TODO add an "int-if"
+ // although mostly unneeded, thanks to the only float being false being 0x0 and 0x80000000 (negative zero)
+ // and entity, string, field values can never have that value
{
prog->xfunction->profile += (st - startst);
st += st->b - 1; // offset the s++
startst = st;
+ // no bounds check needed, it is done when loading progs
#if PRVMRUNAWAYCHECK
if (++jumpcount == 10000000)
{
prog->xfunction->profile += (st - startst);
st += st->a - 1; // offset the s++
startst = st;
+ // no bounds check needed, it is done when loading progs
#if PRVMRUNAWAYCHECK
if (++jumpcount == 10000000)
{
if (!OPA->function)
PRVM_ERROR("NULL function in %s", PRVM_NAME);
+#if PRVMBOUNDSCHECK
+ if(!OPA->function || OPA->function >= (unsigned int)prog->progs->numfunctions)
+ {
+ prog->xfunction->profile += (st - startst);
+ prog->xstatement = st - prog->statements; // we better stay on the previously executed statement
+ PRVM_ERROR("%s CALL outside the program", PRVM_NAME);
+ goto cleanup;
+ }
+#endif
+
newf = &prog->functions[OPA->function];
newf->callcount++;
goto cleanup;
}
#endif
- pr_globals[OPB->_int] = OPA->_float;
+ pr_iglobals[OPB->_int] = OPA->_int;
break;
case OP_GSTOREP_V:
#if PRBOUNDSCHECK
goto cleanup;
}
#endif
- pr_globals[OPB->_int ] = OPA->vector[0];
- pr_globals[OPB->_int+1] = OPA->vector[1];
- pr_globals[OPB->_int+2] = OPA->vector[2];
+ pr_iglobals[OPB->_int ] = OPA->ivector[0];
+ pr_iglobals[OPB->_int+1] = OPA->ivector[1];
+ pr_iglobals[OPB->_int+2] = OPA->ivector[2];
break;
case OP_GADDRESS:
goto cleanup;
}
#endif
- OPC->_float = pr_globals[i];
+ OPC->_int = pr_iglobals[i];
break;
case OP_GLOAD_I:
goto cleanup;
}
#endif
- OPC->_float = pr_globals[OPA->_int];
+ OPC->_int = pr_iglobals[OPA->_int];
break;
case OP_GLOAD_V:
goto cleanup;
}
#endif
- OPC->vector[0] = pr_globals[OPA->_int ];
- OPC->vector[1] = pr_globals[OPA->_int+1];
- OPC->vector[2] = pr_globals[OPA->_int+2];
+ OPC->ivector[0] = pr_iglobals[OPA->_int ];
+ OPC->ivector[1] = pr_iglobals[OPA->_int+1];
+ OPC->ivector[2] = pr_iglobals[OPA->_int+2];
break;
case OP_BOUNDCHECK: