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

Diff Legend

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