Complete.Org: Mailing Lists: Archives: freeciv-dev: July 2006:
[Freeciv-Dev] No check for player!=NULL in player.c diplomatic state fun 		Home

[Freeciv-Dev] No check for player!=NULL in player.c diplomatic state fun

[Top] [All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index] [Thread Index]
To: freeciv-dev@xxxxxxxxxxx
Subject: [Freeciv-Dev] No check for player!=NULL in player.c diplomatic state functions
From: Egor Vyscrebentsov <evyscr@xxxxxxxxx>
Date: Wed, 12 Jul 2006 22:14:54 +0400 
Good daytime!

common/player.c:pplayers_at_war() does not check players to be not
NULLed. (pplayer_get_diplstate(), other pplayers_*()
diplomatic state functions and gives_shared_vision don't check too.)

I'm not sure this is a bug, because NULLed player should not be passed
as argument. But what about assert for this case?

-- 
Thanks, evyscr
  who plays with global observer and editing mode.
[Prev in Thread] Current Thread [Next in Thread]