Oops on invalid actor and victim arguments in nreport()

Replaces the existing, silent recovery from invalid victim argument.

diff --git a/src/lib/subs/nreport.c b/src/lib/subs/nreport.c
index 91ee47832482fd6ca65e329b36f63025b2f3ea33..13216b88bf31dfb5a244590333d7577273ae8178 100644 (file)
--- a/src/lib/subs/nreport.c
+++ b/src/lib/subs/nreport.c
@@ -59,12 +59,13 @@ void
 nreport(natid actor, int event, natid victim, int times)
 {
     int nice;
 nreport(natid actor, int event, natid victim, int times)
 {
     int nice;

-    struct natstr *natp;

     struct newscache *ncp;
 
     if (CANT_HAPPEN((unsigned)event > N_MAX_VERB
                    || rpt[event].r_newstory[0] == rpt[0].r_newstory[0]))
        return;
     struct newscache *ncp;
 
     if (CANT_HAPPEN((unsigned)event > N_MAX_VERB
                    || rpt[event].r_newstory[0] == rpt[0].r_newstory[0]))
        return;

+    if (CANT_HAPPEN(actor >= MAXNOC || victim >= MAXNOC))
+       return;

 
     ncp = ncache(actor, event, victim, times);
     putnews(ncp->id, &ncp->news);
 
     ncp = ncache(actor, event, victim, times);
     putnews(ncp->id, &ncp->news);


@@ -83,9 +84,7 @@ nreport(natid actor, int event, natid victim, int times)
        return;
     if (!chance((double)-nice * times / 20.0))
        return;
        return;
     if (!chance((double)-nice * times / 20.0))
        return;

-    if (!(natp = getnatp(victim)))
-       return;
-    if (getrel(natp, actor) < HOSTILE)
+    if (getrel(getnatp(victim), actor) < HOSTILE)

        return;
 
     setrel(victim, actor, HOSTILE);
        return;
 
     setrel(victim, actor, HOSTILE);