ViewVC Help
View File | Revision Log | Show Annotations | Download File
/cvs/deliantra/server/ext/login.ext
(Generate patch)

Comparing deliantra/server/ext/login.ext (file contents):
Revision 1.90 by root, Tue Sep 16 16:03:02 2008 UTC vs.
Revision 1.119 by root, Thu Nov 15 05:54:02 2012 UTC

1#! perl # mandatory 1#! perl # mandatory depends=highscore
2 2
3# login handling 3# login handling
4 4
5use Fcntl; 5use Fcntl;
6use Coro::AIO; 6use Coro::AIO;
7 7
8our $MAX_DISCONNECT_TIME = $cf::CFG{max_disconnect_time} || 3600; 8CONF MAX_DISCONNECT_TIME = 3600;
9
10# paranoia function to overwrite a string-in-place
11sub nuke_str {
12 substr $_[0], 0, (length $_[0]), "x" x length $_[0]
13}
14 9
15sub query { 10sub query {
16 my ($ns, $flags, $text) = @_; 11 my ($ns, $flags, $text) = @_;
17 12
18 my $current = $Coro::current; 13 $ns->query ($flags, $text, Coro::rouse_cb);
19 $ns->query ($flags, $text, sub { $current->ready; $current = $_[0] }); 14 Coro::rouse_wait
20 Coro::schedule while ref $current;
21
22 $current
23} 15}
24 16
25sub can_cleanup { 17sub can_cleanup {
26 my ($pl, $mtime) = @_; 18 my ($pl, $mtime) = @_;
27 19
62 my $m = $ob->map 54 my $m = $ob->map
63 or return; 55 or return;
64 my $x = $ob->x; 56 my $x = $ob->x;
65 my $y = $ob->y; 57 my $y = $ob->y;
66 58
59 # never happens normally, but helps when shell users make mistakes
60 $m->linkable
61 or return 1;
62
67# return 0;#d# 63# return 0;#d#
68# warn join ":", $m->at ($x, $y);#d# 64# warn join ":", $m->at ($x, $y);#d#
69# warn "FOO$m { ".scalar ($m->at ($x, $y))." }\n"; 65# warn "FOO$m { ".scalar ($m->at ($x, $y))." }\n";
70# return 0; 66# return 0;
71 67
73} 69}
74 70
75sub enter_map { 71sub enter_map {
76 my ($pl) = @_; 72 my ($pl) = @_;
77 73
78 warn $pl->ob->name, ": enter map 1\n";#d#
79 my $ob = $pl->ob; 74 my $ob = $pl->ob;
80 75
81 my ($map, $x, $y) 76 my ($map, $x, $y)
82 = $ob->{_link_pos} 77 = $ob->{_link_pos}
83 ? @{delete $ob->{_link_pos}} 78 ? @{delete $ob->{_link_pos}}
84 : ($pl->maplevel, $ob->x, $ob->y); 79 : ($pl->maplevel, $ob->x, $ob->y);
85 80
86 warn $pl->ob->name, ": enter map 2\n";#d#
87 $ob->enter_link; 81 $ob->enter_link;
88 warn $pl->ob->name, ": enter map 3\n";#d#
89 82
90 my $m = cf::map::find $map; 83 my $m = cf::map::find $map;
91 my $time = delete $pl->{unclean_save}; 84 my $time = delete $pl->{unclean_save};
92 warn $pl->ob->name, ": enter map 4\n";#d#
93 85
94 if ($time && $m) { 86 if ($time && $m) {
95 if ($time < $m->{instantiate_time}) { 87 if ($time < $m->{instantiate_time}) {
96 # the map was reset in the meantime 88 # the map was reset in the meantime
97 my $age = $cf::RUNTIME - $time; 89 my $age = $cf::RUNTIME - $time;
98 90
99 warn $ob->name, " map reset after logout, logout age $age (>= $MAX_DISCONNECT_TIME)\n";#d# 91 cf::info $ob->name, " map reset after logout, logout age $age (>= $MAX_DISCONNECT_TIME)\n";#d#
100 92
101 if ($age >= $MAX_DISCONNECT_TIME) { 93 if ($age >= $MAX_DISCONNECT_TIME) {
102 $ob->message ( 94 $ob->message (
103 "You didn't use a bed to reality to leave this realm, leaving your body in great danger. " 95 "You didn't use a bed to reality to leave this realm, leaving your body in great danger. "
104 . "Unfortunately, nobody was near to help you when the monsters arrived to eat you. " 96 . "Unfortunately, nobody was near to help you when the monsters arrived to eat you. "
130 cf::NDI_RED 122 cf::NDI_RED
131 ); 123 );
132 } 124 }
133 } 125 }
134 126
135 warn $pl->ob->name, ": enter map 5\n";#d#
136 #$ob->goto ($map, $x, $y); 127 $ob->goto ($map, $x, $y);
137 $ob->goto ($map, $x, $y, sub { 128}
138 warn $pl->ob->name, ": enter map check\n";#d# 129
139 $_[0] 130sub encode_password($) {
140 }, sub { 131 "!" . unpack "H*", $_[0]
141 warn $pl->ob->name, ": enter map done\n";#d# 132}
133
134sub compare_password($$) {
135 my ($pass, $token) = @_;
136
137 if ($token =~ /!!(.*)/) {
138 return +(substr $pass, 0, 8) eq pack "H*", $1;
139 } elsif ($token =~ /!(.*)/) {
140 return $pass eq pack "H*", $1;
141 } else {
142 return $token eq crypt $pass, $token;
142 }); 143 }
143 warn $pl->ob->name, ": enter map 6\n";#d#
144
145} 144}
146 145
147# delete a player directory, be non-blocking AND synchronous... 146# delete a player directory
148# (thats hard, so we crap out and fork).
149sub nuke_playerdir { 147sub nuke_playerdir {
150 my ($user) = @_; 148 my ($user) = @_;
151 149
150 my $lock = cf::lock_acquire "ext::login::nuke_playerdir";
151
152 my $temp = "$PLAYERDIR/~$Coro::current~deleting~"; 152 my $temp = "$PLAYERDIR/~$Coro::current~deleting~";
153
154 cf::fork_call {
155 rename "$PLAYERDIR/$user", $temp; 153 aio_rename "$PLAYERDIR/$user", $temp;
156 system "rm", "-rf", $temp; 154 IO::AIO::aio_rmtree $temp;
155}
156
157sub login_done {
158 my ($pl) = @_;
159
160 if (0 < Coro::AIO::aio_load "$cf::CONFDIR/motd", my $motd) {
161 $pl->ns->send_msg ("c/motd" => $motd, cf::NDI_CLEAR);
157 }; 162 }
163}
164
165sub chargen {
166 my ($ns, $user, $pass) = @_;
167
168 # the rest of this function is character creation
169 $Coro::current->{desc} = "addme($user) chargen";
170
171 # just to make sure nothing is left over
172 # normally, nothing is there.
173 nuke_playerdir $user;
174
175 my $pass2 = query $ns, cf::CS_QUERY_HIDEINPUT, "Please type your password again.";
176
177 if ($pass2 ne $pass) {
178 $ns->send_drawinfo (
179 "The passwords do not match, please try again.",
180 cf::NDI_RED
181 );
182 Coro::Timer::sleep 0.5;
183 next;
184 }
185
186 my $pl = cf::player::new $user;
187 $pl->password (encode_password $pass);
188 $pl->connect ($ns);
189 my $ob = $pl->ob;
190
191 $ob->goto ($pl->maplevel, $ob->x, $ob->y);
192
193 while () {
194 $ob->update_stats;
195 $pl->save_stats;
196
197 my $res = query $ns, cf::CS_QUERY_SINGLECHAR,
198 "[y] to roll new stats [n] to use stats\n[1-7] [1-7] to swap stats.\nRoll again (y/n/1-7)?";
199
200 if ($res =~ /^[Nn]/) {
201 last;
202 } elsif ($res > 0 && $res <= 7) {
203 my $swap = query $ns, cf::CS_QUERY_SINGLECHAR, "Swap stat with (will not roll new stats) [1-7]?";
204
205 if ($swap > 0 && $swap <= 7) {
206 $ob->swap_stats ($res - 1, $swap - 1);
207 }
208 } else {
209 $ob->roll_stats;
210 }
211
212 Coro::Timer::sleep 0.05;
213 }
214
215 $ob->set_animation (2);
216 $ob->add_statbonus;
217
218 while () {
219 $ns->send_msg ("chargen-race-title", ucfirst $pl->title, -1);
220 my $msg = $ob->msg;
221 $msg =~ s/(?<=\S)\n(?=\S)/ /g;
222 $ns->send_msg ("chargen-race-description", $msg, cf::NDI_BLUE);
223
224 my $res = query $ns, cf::CS_QUERY_SINGLECHAR,
225 "Now choose a character.\nPress any key to change outlook.\nPress `d' when you're pleased.\n";
226
227 last if $res =~ /[dD]/;
228
229 $pl->chargen_race_next;
230 Coro::Timer::sleep 0.05;
231 }
232
233 # create the playerdir, if necessary, as chargen_race_done did it before
234 # presumably because of unique maps
235 aio_mkdir playerdir $pl, 0770;
236 $pl->chargen_race_done;
237
238 while () {
239 my $res = query $ns, cf::CS_QUERY_SINGLECHAR,
240 "Now choose a gender.\nPress 'f' to become female, and 'm' to become male.\n";
241
242 if ($res =~ /^[fF]/) {
243 $pl->gender (1);
244 last;
245 } elsif ($res =~ /^[mM]/) {
246 $pl->gender (0);
247 last;
248 }
249 Coro::Timer::sleep 0.05;
250 }
251
252 $ob->reply (undef, "Welcome to Deliantra!");
253
254 # XXX: Workaround for delayed client ext protocol handshake
255 $pl->esrv_new_player;
256
257 delete $pl->{deny_save};
158} 258}
159 259
160cf::client->attach (on_addme => sub { 260cf::client->attach (on_addme => sub {
161 my ($ns) = @_; 261 my ($ns) = @_;
162
163 if (!$ns->facecache)
164 {
165 $ns->send_drawinfo (<<EOF, cf::NDI_RED);
166
167
168***
169*** WARNING:
170*** Your client does not support face/image caching,
171*** or it has been disabled. Face caching is mandatory
172*** so please enable it or use a newer client.
173***
174*** Look at your client preferences:
175***
176*** deliantra: all known versions automatically enable the facecache.
177*** cfclient: use the -cache commandline option.
178*** cfclient: map will not redraw automatically (bug).
179*** gcfclient: use -cache commandline option, or enable
180*** gcfclient: Client => Configure => Map & Image => Cache Images.
181*** jcrossclient: your client is broken, use deliantra or gcfclient.
182***
183***
184EOF
185 if ($ns->version =~ /jcrossclient/) {
186 # let them, for now
187 } else {
188 $ns->flush;
189 return $ns->destroy;
190 }
191
192 # $ns->facecache = true;
193 }
194
195 if ($ns->mapmode < cf::Map1aCmd) {
196 $ns->send_drawinfo (<<EOF, cf::NDI_RED);
197
198
199***
200*** WARNING:
201*** Your client is too old. Please upgrade to a newer version.
202EOF
203
204 $ns->flush;
205 return $ns->destroy;
206 }
207 262
208 $ns->pl and return $ns->destroy; 263 $ns->pl and return $ns->destroy;
209 264
210 $ns->async (sub { 265 $ns->async (sub {
211 $Coro::current->{desc} = "addme init"; 266 $Coro::current->{desc} = "addme init";
230 $ns->send_drawinfo ( 285 $ns->send_drawinfo (
231 "That username is currently used in another login session. " 286 "That username is currently used in another login session. "
232 . "Chose another, or wait till the other session has ended.", 287 . "Chose another, or wait till the other session has ended.",
233 cf::NDI_RED 288 cf::NDI_RED
234 ); 289 );
235 } elsif ($user =~ /^[a-zA-Z0-9][a-zA-Z0-9\-_]{2,17}\z/) { 290 } elsif ($user =~ /^[a-zA-Z0-9][a-zA-Z0-9\-_]{2,19}\z/) {
236 last; 291 last;
237 } else { 292 } else {
238 $ns->send_drawinfo ( 293 $ns->send_drawinfo (
239 "Your username contains illegal characters " 294 "Your username contains illegal characters "
240 . "(only a-z, A-Z and 0-9 are allowed), " 295 . "(only a-z, A-Z and 0-9 are allowed), "
241 . "or is not between 3 and 18 characters in length.", 296 . "or is not between 3 and 20 characters in length.",
242 cf::NDI_RED 297 cf::NDI_RED
243 ); 298 );
244 } 299 }
245 Coro::Timer::sleep 0.4; 300 Coro::Timer::sleep 0.4;
246 } 301 }
285 340
286 # try to read the user file and check the password 341 # try to read the user file and check the password
287 if (my $pl = cf::player::find $user) { 342 if (my $pl = cf::player::find $user) {
288 aio_stat $pl->path and next; 343 aio_stat $pl->path and next;
289 my $mtime = (stat _)[9]; 344 my $mtime = (stat _)[9];
290 my $hash = $pl->password; 345 my $token = $pl->password;
291 346
292 if ($cf::CFG{ext_login_nocheck} or $hash eq crypt $pass, $hash) { 347 if ($cf::CFG{ext_login_nocheck} or compare_password $pass, $token) {
293 nuke_str $pass; 348 # player exists and passwords match - we can proceed
349
350 $pl->password (encode_password $pass); # make sure we store the new encoding #d#
294 # password matches, wonderful 351 # password matches, wonderful
295 my $pl = cf::player::find $user or next; 352 my $pl = cf::player::find $user or next;
296 $pl->connect ($ns); 353 $pl->connect ($ns);
297 $pl->ob->flag (cf::FLAG_DEBUG, 1);#d# temp
298 enter_map $pl; 354 enter_map $pl;
355 login_done $pl;
299 last; 356 return;
300 } elsif (can_cleanup $pl, $mtime) { 357 } elsif (can_cleanup $pl, $mtime) {
301 Coro::Timer::sleep 1; 358 Coro::Timer::sleep 1;
302 359
303 $ns->send_drawinfo ( 360 $ns->send_drawinfo (
304 "Player exists, but password does not match. If this is your account, " 361 "Player exists, but password does not match. If this is your account, "
305 . "please try again. If not, you can now decide to take over this account " 362 . "please try again. If not, you can now decide to take over this account "
306 . "because it has not been in-use for some time.", 363 . "because it has not been in-use for some time.",
307 cf::NDI_RED 364 cf::NDI_RED
308 ); 365 );
309 366
310 #TODO: nuke_str
311 (query $ns, cf::CS_QUERY_SINGLECHAR, "Delete existing account and create a new one (Y/N)?") =~ /^[yY]/ 367 (query $ns, cf::CS_QUERY_SINGLECHAR, "Delete existing account and create a new one (Y/N)?") =~ /^[yY]/
312 or next; 368 or next;
313 369
314 # check if the file hasn't changed 370 # check if the file hasn't changed
315 aio_stat cf::player::path $user and next; 371 aio_stat cf::player::path $user and next;
317 373
318 $pl->quit_character; 374 $pl->quit_character;
319 375
320 # fall through to creation 376 # fall through to creation
321 } else { 377 } else {
322 nuke_str $pass;
323
324 Coro::Timer::sleep 1; 378 Coro::Timer::sleep 1;
325 379
326 $ns->send_drawinfo ( 380 $ns->send_drawinfo (
327 "Wrong username or password. Please try again " 381 "Wrong username or password. Please try again "
328 . "(check for Numlock and other semi-obvious error sources).", 382 . "(check for Numlock and other semi-obvious error sources).",
330 ); 384 );
331 next; 385 next;
332 } 386 }
333 } else { 387 } else {
334 # unable to load the playerfile: 388 # unable to load the playerfile:
335 # check wether the player dir exists, which means the file is corrupted or 389 # check whether the player dir exists, which means the file is corrupted or
336 # something very similar. 390 # something very similar.
337 if (!aio_stat cf::player::playerdir $user) { 391 if (!aio_stat cf::player::playerdir $user) {
338 $ns->send_drawinfo ( 392 $ns->send_drawinfo (
339 "Unable to retrieve this player. It might be a locked or broken account. " 393 "Unable to retrieve this player. It might be a locked or broken account. "
340 . "If this is your account, ask a dungeon master for assistance. " 394 . "If this is your account, ask a dungeon master for assistance. "
343 ); 397 );
344 next; 398 next;
345 } 399 }
346 } 400 }
347 401
348 # the rest of this function is character creation
349 $Coro::current->{desc} = "addme($user) chargen";
350
351 # just to make sure nothing is left over
352 nuke_playerdir $user;
353
354 my $pass2 = query $ns, cf::CS_QUERY_HIDEINPUT, "Please type your password again.";
355
356 if ($pass2 ne $pass) {
357 nuke_str $pass;
358 nuke_str $pass2;
359 $ns->send_drawinfo (
360 "The passwords do not match, please try again.",
361 cf::NDI_RED
362 );
363 Coro::Timer::sleep 0.5;
364 next;
365 }
366
367 nuke_str $pass2;
368
369 my $pl = cf::player::new $user;
370 $pl->password (crypt $pass, join '', ('.', '/', 0..9, 'A'..'Z', 'a'..'z')[rand 64, rand 64]);
371 nuke_str $pass;
372 $pl->connect ($ns);
373 my $ob = $pl->ob;
374
375 $ob->goto ($pl->maplevel, $ob->x, $ob->y);
376
377 while () {
378 $ob->update_stats;
379 $pl->save_stats;
380
381 my $res = query $ns, cf::CS_QUERY_SINGLECHAR,
382 "[y] to roll new stats [n] to use stats\n[1-7] [1-7] to swap stats.\nRoll again (y/n/1-7)?";
383
384 if ($res =~ /^[Nn]/) {
385 last;
386 } elsif ($res > 0 && $res <= 7) {
387 my $swap = query $ns, cf::CS_QUERY_SINGLECHAR, "Swap stat with (will not roll new stats) [1-7]?";
388
389 if ($swap > 0 && $swap <= 7) {
390 $ob->swap_stats ($res - 1, $swap - 1);
391 }
392 } else {
393 $ob->roll_stats;
394 }
395
396 Coro::Timer::sleep 0.05;
397 }
398
399 $ob->set_animation (2);
400 $ob->add_statbonus;
401
402 while () {
403 $ns->send_msg ("chargen-race-title", ucfirst $pl->title, -1);
404 my $msg = $ob->msg;
405 $msg =~ s/(?<=\S)\n(?=\S)/ /g;
406 $ns->send_msg ("chargen-race-description", $msg, cf::NDI_BLUE);
407
408 my $res = query $ns, cf::CS_QUERY_SINGLECHAR,
409 "Now choose a character.\nPress any key to change outlook.\nPress `d' when you're pleased.\n";
410
411 last if $res =~ /[dD]/;
412
413 $pl->chargen_race_next;
414 Coro::Timer::sleep 0.2;
415 }
416
417 # create the playerdir, if necessary, as chargen_race_done did it before
418 # presumably because of unique maps
419 aio_mkdir playerdir $pl, 0770;
420 $pl->chargen_race_done;
421
422 while () {
423 my $res = query $ns, cf::CS_QUERY_SINGLECHAR,
424 "Now choose a gender.\nPress 'f' to become female, and 'm' to become male.\n";
425
426 if ($res =~ /^[fF]/) {
427 $pl->gender (1);
428 last;
429 } elsif ($res =~ /^[mM]/) {
430 $pl->gender (0);
431 last;
432 }
433 Coro::Timer::sleep 0.2;
434 }
435
436 $ob->reply (undef, "Welcome to Deliantra!");
437
438 delete $pl->{deny_save};
439
440 last; 402 last;
441 } 403 }
404
405 # lock again, too layz to make this nicer
406 local $cf::LOGIN_LOCK{$user} = 1;
407 chargen $ns, $user, $pass;
408 login_done $ns->pl;
442 }); 409 });
443}); 410});
444 411
412cf::register_command password => sub {
413 my ($pl, $arg) = @_;
414
415 unless ($pl->flag (cf::FLAG_WIZ)) {
416 $pl->message (
417 "The password can currently only changed by a DM.",
418 cf::NDI_UNIQUE | cf::NDI_REPLY);
419 return;
420 }
421
422 my (@args) = split /\s+/, $arg;
423 my ($player, $new_pw) = @args;
424
425 if ($pl->flag (cf::FLAG_WIZ) && $player eq '') {
426 $pl->message (
427 "Usage: password <player> [<new password>]",
428 cf::NDI_UNIQUE | cf::NDI_REPLY);
429 return;
430 }
431
432 if ($new_pw eq '') {
433 $new_pw =
434 join '',
435 map { ('.', '/', 0..9, 'A'..'Z', 'a'..'z')[(cf::rndm 64)] }
436 1..9;
437 }
438
439 cf::async {
440 my $plc = cf::player::find $player;
441 if ($plc) {
442 $plc->password (encode_password $new_pw);
443 $pl->message (
444 "Ok, changed password of '$player' to '$new_pw'!",
445 cf::NDI_UNIQUE | cf::NDI_RED | cf::NDI_REPLY);
446 } else {
447 $pl->message (
448 "Fail! Couldn't set password for '$player', "
449 . "he doesn't seem to exist!",
450 cf::NDI_UNIQUE | cf::NDI_RED | cf::NDI_REPLY);
451 }
452 };
453};
454
445cf::register_command quit => sub { 455cf::register_command quit => sub {
446 my ($ob, $arg) = @_; 456 my ($ob, $arg) = @_;
447 457
448 $ob->reply (undef, 458 $ob->send_msg (undef,
449 "Quitting will delete your character PERMANENTLY: It will be gone forever and any progress will be lost. " 459 "Quitting will delete your character PERMANENTLY: It will be gone forever and any progress will be lost. "
450 . "If you are sure you want to do this, then use the quit_character command instead of quit.", 460 . "If you are sure you want to do this, then use the quit_character command instead of quit.",
451 cf::NDI_UNIQUE | cf::NDI_RED); 461 cf::NDI_UNIQUE | cf::NDI_RED | cf::NDI_REPLY);
452}; 462};
453 463
454cf::register_command quit_character => sub { 464cf::register_command quit_character => sub {
455 my ($ob, $arg) = @_; 465 my ($ob, $arg) = @_;
456 466
457 my $pl = $ob->contr; 467 my $pl = $ob->contr;
458 468
459 $pl->ns->query (cf::CS_QUERY_SINGLECHAR, "Do you want to PERMANENTLY delete your character and all associated data (y/n)?", sub { 469 $pl->ns->query (cf::CS_QUERY_SINGLECHAR, "Do you want to PERMANENTLY delete your character and all associated data (y/n)?", sub {
460 if ($_[0] !~ /^[yY]/) { 470 if ($_[0] !~ /^[yY]/) {
461 $ob->reply (undef, 471 $ob->send_msg (undef, "Ok, not not quitting then.", cf::NDI_UNIQUE | cf::NDI_RED | cf::NDI_REPLY);
462 "Ok, not not quitting then.",
463 cf::NDI_UNIQUE | cf::NDI_RED);
464 } else { 472 } else {
465 $ob->reply (undef, 473 $ob->send_msg (undef, "Ok, quitting, hope to see you again.", cf::NDI_UNIQUE | cf::NDI_RED | cf::NDI_REPLY);
466 "Ok, quitting, hope to see you again.", 474 cf::async {
467 cf::NDI_UNIQUE | cf::NDI_RED);
468 $pl->ns->flush;
469 cf::async { $pl->quit_character }; 475 $pl->quit_character;
476 };
470 } 477 }
471 }); 478 });
472}; 479};
473 480
474cf::object->attach ( 481cf::object->attach (
480 487
481 my $pl = $ob->contr; 488 my $pl = $ob->contr;
482 489
483 # update respawn position 490 # update respawn position
484 $pl->savebed ($bed->map->path, $bed->x, $bed->y); 491 $pl->savebed ($bed->map->path, $bed->x, $bed->y);
485 cf::async { $pl->save };
486 492
493 cf::async {
487 my $killer = cf::arch::get "killer_logout"; $pl->killer ($killer); $killer->destroy; 494 my $killer = cf::arch::get "killer_logout"; $pl->killer ($killer); $killer->destroy;
488 $ob->check_score; 495 ext::highscore::check $ob;
489 496
490 $ob->reply (undef, "In the future, you will wake up here when you die."); 497 $pl->save;
491 498
499 $ob->send_msg ($cf::SAY_CHANNEL => "In the future, you will wake up here when you die.", cf::NDI_DEF | cf::NDI_REPLY);
500
501 my $ns = $pl->ns
502 or return;
503
492 $pl->ns->query (cf::CS_QUERY_SINGLECHAR, "Do you want to continue playing (y/n)?", sub { 504 $ns->query (cf::CS_QUERY_SINGLECHAR, "Do you want to continue playing (y/n)?", sub {
493 if ($_[0] !~ /^[yY]/) { 505 if ($_[0] !~ /^[yY]/) {
494 $pl->invoke (cf::EVENT_PLAYER_LOGOUT, 1); 506 $pl->invoke (cf::EVENT_PLAYER_LOGOUT, 1);
495 $pl->deactivate; 507 $pl->deactivate;
496 $pl->ns->destroy; 508 $pl->ns->destroy;
497 } else { 509 }
498 cf::async { $pl->save };
499 } 510 });
500 }); 511 };
501 }, 512 },
502); 513);
503 514
504cf::player->attach ( 515cf::player->attach (
505 on_login => sub { 516 on_login => sub {
520 unless safe_spot $pl; 531 unless safe_spot $pl;
521 } 532 }
522 }, 533 },
523); 534);
524 535
525#############################################################################
526
527our $SCHEDULE_INTERVAL = $cf::CFG{player_schedule_interval} || 10; # time the player scheduler sleeps between runs
528our $SAVE_TIMEOUT = $cf::CFG{player_save_interval} || 20; # save players every n seconds
529
530our $SCHEDULER = cf::async_ext {
531 $Coro::current->{desc} = "player scheduler";
532
533 while () {
534 Coro::EV::timer_once $SCHEDULE_INTERVAL;
535
536 # this weird form of iteration over values is used because
537 # the hash changes underneath us frequently, and for
538 # keeps a direct reference to the value without (in 5.8 perls)
539 # keeping a reference, so this is prone to crashes or worse.
540 my @players = keys %cf::PLAYER;
541 for (@players) {
542 my $pl = $cf::PLAYER{$_}
543 or next;
544 $pl->valid or next;
545
546 eval {
547 if ($pl->{last_save} + $SAVE_TIMEOUT <= $cf::RUNTIME) {
548 $pl->save;
549
550 unless ($pl->active || $pl->ns) {
551 # check refcounts, this is tricky and needs to be adjusted to fit server internals
552 my $ob = $pl->ob;
553
554 my $pl_ref = $pl->refcnt_cnt;
555 my $ob_ref = $ob->refcnt_cnt;
556
557 ## pl_ref == $pl + ob->contr + %cf::PLAYER
558 ## ob_ref == $ob + pl->observe + simply being an object
559 if ($pl_ref == 3 && $ob_ref == 3) {
560 warn "player-scheduler destroy ", $ob->name;#d#
561
562 # remove from sight and get fresh "copies"
563 $pl = delete $cf::PLAYER{$ob->name};
564 $ob = $pl->ob;
565
566 $pl->destroy; # destroys $ob
567 } else {
568 my $a_ = $pl->refcnt;#d#
569 my $b_ = $ob->refcnt;#d#
570
571 warn "player-scheduler refcnt ", $ob->name, " pl $pl_ref/3 ob $ob_ref/3 (C pl $a_/1 ob $b_/2)\n";#d#
572 }
573 }
574 }
575 };
576 warn $@ if $@;
577 cf::cede_to_tick;
578 };
579 }
580};
581
582$SCHEDULER->prio (1);
583

Diff Legend

Removed lines
+ Added lines
< Changed lines
> Changed lines