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.64 by root, Sat Jul 14 19:27:59 2007 UTC vs.
Revision 1.118 by root, Thu Nov 8 00:16:07 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 9
10# paranoia function to overwrite a string-in-place 10# paranoia function to overwrite a string-in-place
11sub nuke_str { 11sub nuke_str {
12 substr $_[0], 0, (length $_[0]), "x" x length $_[0] 12 substr $_[0], 0, (length $_[0]), "x" x length $_[0]
13} 13}
14 14
15sub query { 15sub query {
16 my ($ns, $flags, $text) = @_; 16 my ($ns, $flags, $text) = @_;
17 17
18 my $current = $Coro::current; 18 $ns->query ($flags, $text, Coro::rouse_cb);
19 $ns->query ($flags, $text, sub { $current->ready; $current = $_[0] }); 19 Coro::rouse_wait
20 Coro::schedule while ref $current;
21
22 $current
23} 20}
24 21
25sub can_cleanup { 22sub can_cleanup {
26 my ($pl, $mtime) = @_; 23 my ($pl, $mtime) = @_;
27 24
… …
52 ); 49 );
53 50
54 1 51 1
55} 52}
56 53
54sub safe_spot($) {
55 my ($pl) = @_;
56
57 my $ob = $pl->ob;
58
59 my $m = $ob->map
60 or return;
61 my $x = $ob->x;
62 my $y = $ob->y;
63
64 # never happens normally, but helps when shell users make mistakes
65 $m->linkable
66 or return 1;
67
68# return 0;#d#
69# warn join ":", $m->at ($x, $y);#d#
70# warn "FOO$m { ".scalar ($m->at ($x, $y))." }\n";
71# return 0;
72
73 scalar grep $_->type == cf::SAVEBED, $m->at ($x, $y)
74}
75
57sub enter_map { 76sub enter_map {
58 my ($pl) = @_; 77 my ($pl) = @_;
59 78
60 my $ob = $pl->ob; 79 my $ob = $pl->ob;
61 80
… …
64 ? @{delete $ob->{_link_pos}} 83 ? @{delete $ob->{_link_pos}}
65 : ($pl->maplevel, $ob->x, $ob->y); 84 : ($pl->maplevel, $ob->x, $ob->y);
66 85
67 $ob->enter_link; 86 $ob->enter_link;
68 87
88 my $m = cf::map::find $map;
69 if (my $time = delete $pl->{unclean_save}) { 89 my $time = delete $pl->{unclean_save};
70 if (my $m = cf::map::find $map) { 90
91 if ($time && $m) {
71 if ($time < $m->{instantiate_time}) { 92 if ($time < $m->{instantiate_time}) {
72 # the map was reset in the meantime 93 # the map was reset in the meantime
73 my $age = $cf::RUNTIME - $time; 94 my $age = $cf::RUNTIME - $time;
74 95
75 warn $ob->name, " map reset after logout, logout age $age (>= $MAX_DISCONNECT_TIME)\n";#d# 96 cf::info $ob->name, " map reset after logout, logout age $age (>= $MAX_DISCONNECT_TIME)\n";#d#
76 97
77 # for now, just go back to savebed
78 ($map, $x, $y) = $pl->savebed;
79
80 if ($age >= $MAX_DISCONNECT_TIME) { 98 if ($age >= $MAX_DISCONNECT_TIME) {
81 $ob->message (
82 "You didn't use a bed to reality to leave this realm, leaving your body in great danger. "
83 . "Unfortunately, nobody was near to help you when the monsters arrived to eat you. "
84 . "Maybe you can find comfort in the thought that your body was quite satisfying in taste... "
85 . "H<You disconnected too long without having used a savebed.>",
86 cf::NDI_RED
87 );
88 # kill them.
89 # reminds me of the famous badness 10000 syndrome...
90 $ob->stats->hp (-10000); #] if they survive this they deserved to live
91 } else {
92 $ob->message (
93 "You didn't use a bed to reality to leave this realm, leaving your body in great danger. "
94 . "Fortunately, some friendly dwellers found you, checked your passport, and brought you to safety. "
95 . "Better use a savebed next time, much worse things could have happened... "
96 . "H<You disconnected without having used a savebed. When you do that for too long, you might die.>",
97 cf::NDI_RED
98 );
99 }
100 } else {
101 $ob->message ( 99 $ob->message (
102 "You didn't use a bed to reality to leave this realm. This is very dangerous, " 100 "You didn't use a bed to reality to leave this realm, leaving your body in great danger. "
103 . "as lots of things could happen when you leave by other means, such as cave-ins, " 101 . "Unfortunately, nobody was near to help you when the monsters arrived to eat you. "
104 . "or monsters suddenly snapping your body. Better use a savebed next time. " 102 . "Maybe you can find comfort in the thought that your body was quite satisfying in taste... "
105 . "H<Always apply a bed of reality to disconnect from the server.>", 103 . "H<You disconnected too long without having used a savebed.>",
106 cf::NDI_RED 104 cf::NDI_RED
107 ); 105 );
106 # kill them.
107 # reminds me of the famous badness 10000 syndrome...
108 $ob->stats->hp (-10000); #] if they survive this they deserved to live
109 my $killer = cf::arch::get "killer_login"; $pl->killer ($killer); $killer->destroy;
110 } else {
111 ($map, $x, $y) = $pl->savebed;
112
113 $ob->message (
114 "You didn't use a bed to reality to leave this realm, leaving your body in great danger. "
115 . "Fortunately, some friendly dwellers found you, checked your passport, and brought you to safety. "
116 . "Better use a savebed next time, much worse things could have happened... "
117 . "H<You disconnected without having used a savebed. When you do that for too long, you might die.>",
118 cf::NDI_RED
119 );
120 }
121 } else {
122 $ob->message (
123 "You didn't use a bed to reality to leave this realm. This is very dangerous, "
124 . "as lots of things could happen when you leave by other means, such as cave-ins, "
125 . "or monsters suddenly snapping your body. Better use a savebed next time. "
126 . "H<Always apply a bed of reality to disconnect from the server.>",
127 cf::NDI_RED
108 } 128 );
109 } 129 }
110 } 130 }
111 131
112 $ob->goto ($map, $x, $y); 132 $ob->goto ($map, $x, $y);
113} 133}
114 134
115# delete a player directory, be non-blocking AND synchronous... 135sub encode_password($) {
116# (thats hard, so we crap out and fork). 136# crypt $_[0],
137# join '',
138# ('.', '/', 0..9, 'A'..'Z', 'a'..'z')[(cf::rndm 64), (cf::rndm 64)]
139 "!" . unpack "H*", $_[0]
140}
141
142sub compare_password($$) {
143 my ($pass, $token) = @_;
144
145 if ($token =~ /!!(.*)/) {
146 return +(substr $pass, 0, 8) eq pack "H*", $1;
147 } elsif ($token =~ /!(.*)/) {
148 return $pass eq pack "H*", $1;
149 } else {
150 return $token eq crypt $pass, $token;
151 }
152}
153
154# delete a player directory
117sub nuke_playerdir { 155sub nuke_playerdir {
118 my ($user) = @_; 156 my ($user) = @_;
119 157
158 my $lock = cf::lock_acquire "ext::login::nuke_playerdir";
159
160 my $temp = "$PLAYERDIR/~$Coro::current~deleting~";
120 aio_stat "$PLAYERDIR/$user"; 161 aio_rename "$PLAYERDIR/$user", $temp;
121 system "cd \Q$PLAYERDIR\E " 162 IO::AIO::aio_rmtree $temp;
122 . "&& mv \Q$user\E ~\Q$Coro::current\E~deleting~ 2>/dev/null "
123 . "&& (rm -rf ~\Q$Coro::current\E~deleting~ &)";
124} 163}
125 164
126cf::client->attach (on_addme => sub { 165cf::client->attach (on_addme => sub {
127 my ($ns) = @_; 166 my ($ns) = @_;
128 167
129 if (!$ns->facecache)
130 {
131 $ns->send_drawinfo (<<EOF, cf::NDI_RED);
132
133
134***
135*** WARNING:
136*** Your client does not support face/image caching,
137*** or it has been disabled. Face caching is mandatory
138*** so please enable it or use a newer client.
139***
140*** Look at your client preferences:
141***
142*** CFPlus: all known versions automatically enable the facecache.
143*** cfclient: use the -cache commandline option.
144*** cfclient: map will not redraw automatically (bug).
145*** gcfclient: use -cache commandline option, or enable
146*** gcfclient: Client=>Configure=>Map & Image=>Cache Images.
147*** jcrossclient: your client is broken, use CFPlus or gcfclient.
148***
149***
150EOF
151 if ($ns->version =~ /jcrossclient/) {
152 # let them, for now
153 } else {
154 $ns->flush;
155 return $ns->destroy;
156 }
157
158 # $ns->facecache = true;
159 }
160
161 if ($ns->mapmode < cf::Map1aCmd) {
162 $ns->send_drawinfo (<<EOF, cf::NDI_RED);
163
164
165***
166*** WARNING:
167*** Your client is too old. Please upgrade to a newer version.
168EOF
169
170 $ns->flush;
171 return $ns->destroy;
172 }
173
174 $ns->pl and return $ns->destroy; 168 $ns->pl and return $ns->destroy;
175 169
176 $ns->async (sub { 170 $ns->async (sub {
171 $Coro::current->{desc} = "addme init";
172
177 my ($user, $pass); 173 my ($user, $pass);
178 174
179 $ns->send_packet ("addme_success"); 175 $ns->send_packet ("addme_success");
180 176
181 for (;;) { 177 for (;;) {
… …
186 cf::NDI_BLUE 182 cf::NDI_BLUE
187 ); 183 );
188 184
189 # read username 185 # read username
190 while () { 186 while () {
191 $user = query $ns, 0, "What is your name?\n:"; 187 $user = query $ns, 0, "What is your name? (login names are case-sensitive)\n:";
192 188
193 if ($cf::LOGIN_LOCK{$user}) { 189 if ($cf::LOGIN_LOCK{$user}) {
194 $ns->send_drawinfo ( 190 $ns->send_drawinfo (
195 "That username is currently used in another login session. " 191 "That username is currently used in another login session. "
196 . "Chose another, or wait till the other session has ended.", 192 . "Chose another, or wait till the other session has ended.",
197 cf::NDI_RED 193 cf::NDI_RED
198 ); 194 );
199 } elsif ($user =~ /^[a-zA-Z0-9][a-zA-Z0-9\-_]{2,17}\z/) { 195 } elsif ($user =~ /^[a-zA-Z0-9][a-zA-Z0-9\-_]{2,19}\z/) {
200 last; 196 last;
201 } else { 197 } else {
202 $ns->send_drawinfo ( 198 $ns->send_drawinfo (
203 "Your username contains illegal characters " 199 "Your username contains illegal characters "
204 . "(only a-z, A-Z and 0-9 are allowed), " 200 . "(only a-z, A-Z and 0-9 are allowed), "
205 . "or is not between 3 and 18 characters in length.", 201 . "or is not between 3 and 20 characters in length.",
206 cf::NDI_RED 202 cf::NDI_RED
207 ); 203 );
208 } 204 }
209 Coro::Timer::sleep 0.4; 205 Coro::Timer::sleep 0.4;
210 } 206 }
211 207
212 check_playing $ns, $user and next; 208 check_playing $ns, $user and next;
209
210 $Coro::current->{desc} = "addme($user) pass";
213 211
214 $ns->send_drawinfo ( 212 $ns->send_drawinfo (
215 "Welcome $user, please enter your password now. " 213 "Welcome $user, please enter your password now. "
216 . "New users should now choose a password. " 214 . "New users should now choose a password. "
217 . "Anything your client lets you enter is fine.", 215 . "Anything your client lets you enter is fine.",
… …
241 } 239 }
242 local $cf::LOGIN_LOCK{$user} = 1; 240 local $cf::LOGIN_LOCK{$user} = 1;
243 241
244 check_playing $ns, $user and next; 242 check_playing $ns, $user and next;
245 243
244 $Coro::current->{desc} = "addme($user) check";
245
246 # try to read the user file and check the password 246 # try to read the user file and check the password
247 if (my $pl = cf::player::find $user) { 247 if (my $pl = cf::player::find $user) {
248 aio_stat $pl->path and next; 248 aio_stat $pl->path and next;
249 my $mtime = (stat _)[9]; 249 my $mtime = (stat _)[9];
250 my $hash = $pl->password; 250 my $token = $pl->password;
251 251
252 if ($cf::CFG{ext_login_nocheck} or $hash eq crypt $pass, $hash) { 252 if ($cf::CFG{ext_login_nocheck} or compare_password $pass, $token) {
253 $pl->password (encode_password $pass); # make sure we store the new encoding #d#
253 nuke_str $pass; 254 nuke_str $pass;
254 # password matches, wonderful 255 # password matches, wonderful
255 my $pl = cf::player::find $user or next; 256 my $pl = cf::player::find $user or next;
256 $pl->connect ($ns); 257 $pl->connect ($ns);
257 enter_map $pl; 258 enter_map $pl;
… …
289 ); 290 );
290 next; 291 next;
291 } 292 }
292 } else { 293 } else {
293 # unable to load the playerfile: 294 # unable to load the playerfile:
294 # check wether the player dir exists, which means the file is corrupted or 295 # check whether the player dir exists, which means the file is corrupted or
295 # something very similar. 296 # something very similar.
296 if (!aio_stat cf::player::playerdir $user) { 297 if (!aio_stat cf::player::playerdir $user) {
297 $ns->send_drawinfo ( 298 $ns->send_drawinfo (
298 "Unable to retrieve this player. It might be a locked or broken account. " 299 "Unable to retrieve this player. It might be a locked or broken account. "
299 . "If this is your account, ask a dungeon master for assistance. " 300 . "If this is your account, ask a dungeon master for assistance. "
… …
303 next; 304 next;
304 } 305 }
305 } 306 }
306 307
307 # the rest of this function is character creation 308 # the rest of this function is character creation
309 $Coro::current->{desc} = "addme($user) chargen";
308 310
309 # just to make sure nothing is left over 311 # just to make sure nothing is left over
312 # normally, nothing is there.
310 nuke_playerdir $user; 313 nuke_playerdir $user;
311 314
312 my $pass2 = query $ns, cf::CS_QUERY_HIDEINPUT, "Please type your password again."; 315 my $pass2 = query $ns, cf::CS_QUERY_HIDEINPUT, "Please type your password again.";
313 316
314 if ($pass2 ne $pass) { 317 if ($pass2 ne $pass) {
… …
323 } 326 }
324 327
325 nuke_str $pass2; 328 nuke_str $pass2;
326 329
327 my $pl = cf::player::new $user; 330 my $pl = cf::player::new $user;
328 $pl->password (crypt $pass, join '', ('.', '/', 0..9, 'A'..'Z', 'a'..'z')[rand 64, rand 64]); 331 $pl->password (encode_password $pass);
329 nuke_str $pass; 332 nuke_str $pass;
330 $pl->connect ($ns); 333 $pl->connect ($ns);
331 my $ob = $pl->ob; 334 my $ob = $pl->ob;
332 335
333 $ob->goto ($pl->maplevel, $ob->x, $ob->y); 336 $ob->goto ($pl->maplevel, $ob->x, $ob->y);
… …
349 } 352 }
350 } else { 353 } else {
351 $ob->roll_stats; 354 $ob->roll_stats;
352 } 355 }
353 356
354 Coro::Timer::sleep 0.2; 357 Coro::Timer::sleep 0.05;
355 } 358 }
356 359
357 $ob->set_animation (2); 360 $ob->set_animation (2);
358 $ob->add_statbonus; 361 $ob->add_statbonus;
359 362
360 while () { 363 while () {
361 $ns->send_msg (-1, "chargen-race-title", ucfirst $pl->title); 364 $ns->send_msg ("chargen-race-title", ucfirst $pl->title, -1);
362 my $msg = $ob->msg; 365 my $msg = $ob->msg;
363 $msg =~ s/(?<=\S)\n(?=\S)/ /g; 366 $msg =~ s/(?<=\S)\n(?=\S)/ /g;
364 $ns->send_msg (cf::NDI_BLUE, "chargen-race-description", $msg); 367 $ns->send_msg ("chargen-race-description", $msg, cf::NDI_BLUE);
365 368
366 my $res = query $ns, cf::CS_QUERY_SINGLECHAR, 369 my $res = query $ns, cf::CS_QUERY_SINGLECHAR,
367 "Now choose a character.\nPress any key to change outlook.\nPress `d' when you're pleased.\n"; 370 "Now choose a character.\nPress any key to change outlook.\nPress `d' when you're pleased.\n";
368 371
369 last if $res =~ /[dD]/; 372 last if $res =~ /[dD]/;
370 373
371 $pl->chargen_race_next; 374 $pl->chargen_race_next;
372 Coro::Timer::sleep 0.2; 375 Coro::Timer::sleep 0.05;
373 } 376 }
374 377
378 # create the playerdir, if necessary, as chargen_race_done did it before
379 # presumably because of unique maps
380 aio_mkdir playerdir $pl, 0770;
375 $pl->chargen_race_done; 381 $pl->chargen_race_done;
376 382
377 while () { 383 while () {
378 my $res = query $ns, cf::CS_QUERY_SINGLECHAR, 384 my $res = query $ns, cf::CS_QUERY_SINGLECHAR,
379 "Now choose a gender.\nPress 'f' to become female, and 'm' to become male.\n"; 385 "Now choose a gender.\nPress 'f' to become female, and 'm' to become male.\n";
… …
383 last; 389 last;
384 } elsif ($res =~ /^[mM]/) { 390 } elsif ($res =~ /^[mM]/) {
385 $pl->gender (0); 391 $pl->gender (0);
386 last; 392 last;
387 } 393 }
388 Coro::Timer::sleep 0.2; 394 Coro::Timer::sleep 0.05;
389 } 395 }
390 396
391 $ob->reply (undef, "Welcome to Crossfire!"); 397 $ob->reply (undef, "Welcome to Deliantra!");
398
399 # XXX: Workaround for delayed client ext protocol handshake
400 $pl->esrv_new_player;
392 401
393 delete $pl->{deny_save}; 402 delete $pl->{deny_save};
394 403
395 last; 404 last;
405 }
406
407 if (0 < Coro::AIO::aio_load "$cf::CONFDIR/motd", my $motd) {
408 $ns->send_msg ("c/motd" => $motd, cf::NDI_CLEAR);
396 } 409 }
397 }); 410 });
398}); 411});
399 412
413cf::register_command password => sub {
414 my ($pl, $arg) = @_;
415
416 unless ($pl->flag (cf::FLAG_WIZ)) {
417 $pl->message (
418 "The password can currently only changed by a DM.",
419 cf::NDI_UNIQUE | cf::NDI_REPLY);
420 return;
421 }
422
423 my (@args) = split /\s+/, $arg;
424 my ($player, $new_pw) = @args;
425
426 if ($pl->flag (cf::FLAG_WIZ) && $player eq '') {
427 $pl->message (
428 "Usage: password <player> [<new password>]",
429 cf::NDI_UNIQUE | cf::NDI_REPLY);
430 return;
431 }
432
433 if ($new_pw eq '') {
434 $new_pw =
435 join '',
436 map { ('.', '/', 0..9, 'A'..'Z', 'a'..'z')[(cf::rndm 64)] }
437 1..9;
438 }
439
440 cf::async {
441 my $plc = cf::player::find $player;
442 if ($plc) {
443 $plc->password (encode_password $new_pw);
444 $pl->message (
445 "Ok, changed password of '$player' to '$new_pw'!",
446 cf::NDI_UNIQUE | cf::NDI_RED | cf::NDI_REPLY);
447 } else {
448 $pl->message (
449 "Fail! Couldn't set password for '$player', "
450 . "he doesn't seem to exist!",
451 cf::NDI_UNIQUE | cf::NDI_RED | cf::NDI_REPLY);
452 }
453 };
454};
455
400cf::register_command quit => sub { 456cf::register_command quit => sub {
401 my ($ob, $arg) = @_; 457 my ($ob, $arg) = @_;
402 458
403 $ob->reply (undef, 459 $ob->send_msg (undef,
404 "Quitting will delete your character PERMANENTLY: It will be gone forever and any progress will be lost. " 460 "Quitting will delete your character PERMANENTLY: It will be gone forever and any progress will be lost. "
405 . "If you are sure you want to do this, then use the quit_character command instead of quit.", 461 . "If you are sure you want to do this, then use the quit_character command instead of quit.",
406 cf::NDI_UNIQUE | cf::NDI_RED); 462 cf::NDI_UNIQUE | cf::NDI_RED | cf::NDI_REPLY);
407}; 463};
408 464
409cf::register_command quit_character => sub { 465cf::register_command quit_character => sub {
410 my ($ob, $arg) = @_; 466 my ($ob, $arg) = @_;
411 467
412 my $pl = $ob->contr; 468 my $pl = $ob->contr;
413 469
414 $pl->ns->query (cf::CS_QUERY_SINGLECHAR, "Do you want to PERMANENTLY delete your character and all associated data (y/n)?", sub { 470 $pl->ns->query (cf::CS_QUERY_SINGLECHAR, "Do you want to PERMANENTLY delete your character and all associated data (y/n)?", sub {
415 if ($_[0] !~ /^[yY]/) { 471 if ($_[0] !~ /^[yY]/) {
416 $ob->reply (undef, 472 $ob->send_msg (undef, "Ok, not not quitting then.", cf::NDI_UNIQUE | cf::NDI_RED | cf::NDI_REPLY);
417 "Ok, not not quitting then.",
418 cf::NDI_UNIQUE | cf::NDI_RED);
419 } else { 473 } else {
420 $ob->reply (undef, 474 $ob->send_msg (undef, "Ok, quitting, hope to see you again.", cf::NDI_UNIQUE | cf::NDI_RED | cf::NDI_REPLY);
421 "Ok, quitting, hope to see you again.", 475 cf::async {
422 cf::NDI_UNIQUE | cf::NDI_RED);
423 $pl->ns->flush;
424 cf::async { $pl->quit_character }; 476 $pl->quit_character;
477 };
425 } 478 }
426 }); 479 });
427}; 480};
428 481
429cf::object->attach ( 482cf::object->attach (
… …
435 488
436 my $pl = $ob->contr; 489 my $pl = $ob->contr;
437 490
438 # update respawn position 491 # update respawn position
439 $pl->savebed ($bed->map->path, $bed->x, $bed->y); 492 $pl->savebed ($bed->map->path, $bed->x, $bed->y);
440 cf::async { $pl->save };
441 493
442 $pl->killer ("left"); 494 cf::async {
443 $ob->check_score; 495 my $killer = cf::arch::get "killer_logout"; $pl->killer ($killer); $killer->destroy;
496 ext::highscore::check $ob;
444 497
445 $ob->reply (undef, "In the future, you will wake up here when you die."); 498 $pl->save;
446 499
500 $ob->send_msg ($cf::SAY_CHANNEL => "In the future, you will wake up here when you die.", cf::NDI_DEF | cf::NDI_REPLY);
501
502 my $ns = $pl->ns
503 or return;
504
447 $pl->ns->query (cf::CS_QUERY_SINGLECHAR, "Do you want to continue playing (y/n)?", sub { 505 $ns->query (cf::CS_QUERY_SINGLECHAR, "Do you want to continue playing (y/n)?", sub {
448 if ($_[0] !~ /^[yY]/) { 506 if ($_[0] !~ /^[yY]/) {
449 $pl->invoke (cf::EVENT_PLAYER_LOGOUT, 1); 507 $pl->invoke (cf::EVENT_PLAYER_LOGOUT, 1);
450 $pl->deactivate; 508 $pl->deactivate;
451 $pl->ns->destroy; 509 $pl->ns->destroy;
452 } else { 510 }
453 cf::async { $pl->save };
454 } 511 });
455 }); 512 };
456 }, 513 },
457); 514);
458 515
459cf::player->attach ( 516cf::player->attach (
460 on_login => sub { 517 on_login => sub {
… …
469 526
470 if ($cleanly) { 527 if ($cleanly) {
471 $_->ob->message ("$name left the game.", cf::NDI_DK_ORANGE | cf::NDI_UNIQUE) for cf::player::list; 528 $_->ob->message ("$name left the game.", cf::NDI_DK_ORANGE | cf::NDI_UNIQUE) for cf::player::list;
472 } else { 529 } else {
473 $_->ob->message ("$name uncerimoniously disconnected.", cf::NDI_DK_ORANGE | cf::NDI_UNIQUE) for cf::player::list; 530 $_->ob->message ("$name uncerimoniously disconnected.", cf::NDI_DK_ORANGE | cf::NDI_UNIQUE) for cf::player::list;
474 $pl->{unclean_save} = $cf::RUNTIME; 531 $pl->{unclean_save} = $cf::RUNTIME
532 unless safe_spot $pl;
475 } 533 }
476 }, 534 },
477); 535);
478 536
479#############################################################################
480
481our $SCHEDULE_INTERVAL = 10; # time the player scheduler sleeps between runs
482our $SAVE_TIMEOUT = 20; # save players every n seconds
483
484our $SCHEDULER = cf::async_ext {
485 my $schedule_interval = Coro::Event->timer (after => 1, interval => $SCHEDULE_INTERVAL);
486 while () {
487 $schedule_interval->next;
488
489 # this weird form of iteration over values is used because
490 # the hash changes underneath us frequently, and for
491 # keeps a direct reference to the value without (in 5.8 perls)
492 # keeping a reference, so this is prone to crashes or worse.
493 my @players = keys %cf::PLAYER;
494 for (@players) {
495 my $pl = $cf::PLAYER{$_}
496 or next;
497 $pl->valid or next;
498
499 eval {
500 if ($pl->{last_save} + $SAVE_TIMEOUT <= $cf::RUNTIME) {
501 cf::wait_for_tick_begin;
502 $pl->save;
503
504 unless ($pl->active || $pl->ns) {
505 # check refcounts, this is tricky and needs to be adjusted to fit server internals
506 my $ob = $pl->ob;
507
508 my $pl_ref = $pl->refcnt_cnt;
509 my $ob_ref = $ob->refcnt_cnt;
510
511 ## pl_ref == $pl + ob->contr + %cf::PLAYER
512 ## ob_ref == $ob + pl->observe + simply being an object
513 if ($pl_ref == 3 && $ob_ref == 3) {
514 warn "player-scheduler destroy ", $ob->name;#d#
515
516 # remove from sight and get fresh "copies"
517 $pl = delete $cf::PLAYER{$ob->name};
518 $ob = $pl->ob;
519
520 $pl->destroy; # destroys $ob
521 } else {
522 my $a_ = $pl->refcnt;#d#
523 my $b_ = $ob->refcnt;#d#
524
525 warn "player-scheduler refcnt ", $ob->name, " pl $pl_ref ob $ob_ref (C pl $a_ ob $b_)\n";#d#
526 }
527 }
528 }
529 };
530 warn $@ if $@;
531 Coro::cede;
532 };
533 }
534};
535
536$SCHEDULER->prio (1);
537

Diff Legend

– Removed lines
+ Added lines
< Changed lines
> Changed lines