Login
User Name:

Password:



Register
Forgot your password?
Vote for Us!
parse description bug
Dec 15, 2017, 10:08 pm
By Remcon
Couple bugs
Dec 12, 2017, 5:42 pm
By Remcon
Bug in disarm( )
Nov 12, 2017, 6:54 pm
By GatewaySysop
Bug in will_fall( )
Oct 23, 2017, 1:35 am
By GatewaySysop
Bug in do_zap( ), do_brandish( )
Oct 18, 2017, 1:52 pm
By GatewaySysop
LOP 1.45
Author: Remcon
Submitted by: Remcon
LOP Heroes Edition
Author: Vladaar
Submitted by: Vladaar
Heroes sound extras
Author: Vladaar
Submitted by: Vladaar
6Dragons 4.3
Author: Vladaar
Submitted by: Vladaar
Memwatch
Author: Johan Lindh
Submitted by: Vladaar
Users Online
CommonCrawl, Google, Yahoo!

Members: 0
Guests: 7
Stats
Files
Topics
Posts
Members
Newest Member
477
3,706
19,240
608
LAntorcha
Today's Birthdays
There are no member birthdays today.
Related Links
» SmaugMuds.org » Bugfix Lists » SmaugFUSS Bugfix List » [Bug] The AFK flag check in i...
Forum Rules | Mark all | Recent Posts

[Bug] The AFK flag check in interp.c doesn't verify if the user is a player
< Newer Topic :: Older Topic >

Pages:<< prev 1 next >>
Post is unread #1 Mar 5, 2005, 4:00 pm   Last edited May 11, 2005, 6:44 pm by Samson
Go to the top of the page
Go to the bottom of the page

Samson
Black Hand
GroupAdministrators
Posts3,639
JoinedJan 1, 2002

Bug: The AFK flag check in interp.c doesn't verify if the user is a player.
Danger: low
Found by: gatewaysysop
Fixed by: Samson and gatewaysysop

[interp.c]

Find:

	/*
  * Turn off afk bit when any command performed.
  */
	if ( xIS_SET ( ch->act, PLR_AFK)  && (str_cmp(command, "AFK";)))
	{
     xREMOVE_BIT( ch->act, PLR_AFK );
/*
          act( AT_GREY, "$n is no longer afk.", ch, NULL, NULL, TO_ROOM );
*/
          act( AT_GREY, "$n is no longer afk.", ch, NULL, NULL, TO_CANSEE );
	}


Change it to:

	/*
  * Turn off afk bit when any command performed.
  */
	if ( !IS_NPC(ch) && xIS_SET ( ch->act, PLR_AFK)  && (str_cmp(command, "AFK";)))
	{
     xREMOVE_BIT( ch->act, PLR_AFK );
/*
          act( AT_GREY, "$n is no longer afk.", ch, NULL, NULL, TO_ROOM );
*/
          act( AT_GREY, "$n is no longer afk.", ch, NULL, NULL, TO_CANSEE );
	}


The reason for the extra IS_NPC check is because the PLR_* and ACT_* flag sets share the same ch->act pointer and the PLR_AFK check could potentially be triggered in error by a mob with the right ACT_* flag set.
       
Pages:<< prev 1 next >>