Noch ein Bug in sp_kampfzauber, diesmal ein total anderer

src/common/kernel/combatspells.c

@@ -148,12 +148,12 @@ sp_kampfzauber(fighter * fi, int level, double power, spell * sp)
 	scat(":");
 	battlerecord(b, buf);
 
-	do {
+	while (force>0 && killed < enemies) {
 		dt = select_enemy(b, fi, minrow, maxrow);
 		assert(dt.fighter);
 		--force;
 		killed += terminate(dt, at, AT_COMBATSPELL, damage, false);
-	} while (force && killed < enemies);
+	} 
 
 	sprintf(buf, "%d Personen %s getötet",
 			killed, killed == 1 ? "wurde" : "wurden");

src/common/kernel/eressea.c

@@ -1910,8 +1910,7 @@ firstregion (faction * f)
 		plane * p = rplane(r);
 		for (u = r->units; u; u = u->next) {
 			if (u->faction == f) {
-				f->first = r;
+				return f->first = r;
-				return r;
 			}
 		}
 		if (f->first == r->next)
@@ -1919,21 +1918,17 @@ firstregion (faction * f)
 		for (ru = a_find(r->attribs, &at_travelunit); ru; ru = ru->nexttype) {
 			u = (unit*)ru->data.v;
 			if (u->faction == f) {
-				f->first = r;
+				return f->first = r;
-				return r;
 			}
 		}
 		if (check_leuchtturm(r, f)) {
-			f->first = r;
+			return f->first = r;
-			return r;
 		}
 		if (p && is_watcher(p, f)) {
-			f->first = r;
+			return f->first = r;
-			return r;
 		}
 	}
-	f->first = regions;
+	return f->first = regions;
-	return regions;
 }
 
 void ** blk_list[1024];