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.17 by root, Mon Jan 8 14:11:05 2007 UTC vs.
Revision 1.114 by root, Fri Jan 21 19:45:07 2011 UTC

1#! perl 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
8my $PLAYERDIR = sprintf "%s/%s", cf::localdir, cf::playerdir; 8our $MAX_DISCONNECT_TIME = $cf::CFG{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 ($playerfile, $mtime) = @_; 23 my ($pl, $mtime) = @_;
27 24
28 my $age = time - $mtime; 25 my $age = time - $mtime;
29 my $level = $playerfile =~ /^level (\d+)$/m ? $1 : return; 26 my $level = $pl->ob->level;
30 27
31 ($level <= 3 && $age > 7 * 86400) # 7 days for level 0..3 28 ($level <= 3 && $age > 7 * 86400) # 7 days for level 0..3
32 || ($level <= 9 && $age > 90 * 86400) # 3 months for level 4..9 29 || ($level <= 9 && $age > 90 * 86400) # 3 months for level 4..9
33 || ($level <= 20 && $age > 180 * 86400) # 6 months for level 10..20 30 || ($level <= 20 && $age > 180 * 86400) # 6 months for level 10..20
34 || $age > 700 * 86400 # 2 years for everybody else 31 || $age > 700 * 86400 # 2 years for everybody else
… …
40 return unless cf::player::find_active $user; 37 return unless cf::player::find_active $user;
41 38
42 $ns->send_drawinfo ( 39 $ns->send_drawinfo (
43 "That player is already logged in on this server. " 40 "That player is already logged in on this server. "
44 . "If you want to create a new player, choose another name. " 41 . "If you want to create a new player, choose another name. "
45 . "If you are already a registered player, make sure nobody " 42 . "If you have already a registered, make sure nobody "
46 . "else is using your account at this time. If you lost your conenction " 43 . "else is using your account at this time. If you lost your connection "
47 . "then the server will likely timeout within a minute. If you still " 44 . "then the server will likely timeout within a minute. If you still "
48 . "cannot log-in after a minute, you are still logged in. Make sure " 45 . "cannot log-in after a minute, you are still logged in. Make sure "
49 . "you do not have another client running. If you use windows, reboot, " 46 . "you do not have another client running. If you use windows, reboot, "
50 . "this will fix anything.", 47 . "this will fix anything.",
51 cf::NDI_RED 48 cf::NDI_RED
52 ); 49 );
53 50
54 1 51 1
55} 52}
56 53
57sub check_clean_save { 54sub safe_spot($) {
58 my ($pl) = @_; 55 my ($pl) = @_;
56
57 my $ob = $pl->ob;
59 58
60 unless (delete $pl->{clean_save}) { 59 my $m = $ob->map
61 #d#TODO 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->in_memory == cf::MAP_ACTIVE
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
76sub enter_map {
77 my ($pl) = @_;
78
79 my $ob = $pl->ob;
80
81 my ($map, $x, $y)
82 = $ob->{_link_pos}
83 ? @{delete $ob->{_link_pos}}
84 : ($pl->maplevel, $ob->x, $ob->y);
85
86 $ob->enter_link;
87
88 my $m = cf::map::find $map;
89 my $time = delete $pl->{unclean_save};
90
91 if ($time && $m) {
92 if ($time < $m->{instantiate_time}) {
93 # the map was reset in the meantime
94 my $age = $cf::RUNTIME - $time;
95
96 cf::info $ob->name, " map reset after logout, logout age $age (>= $MAX_DISCONNECT_TIME)\n";#d#
97
98 if ($age >= $MAX_DISCONNECT_TIME) {
99 $ob->message (
100 "You didn't use a bed to reality to leave this realm, leaving your body in great danger. "
101 . "Unfortunately, nobody was near to help you when the monsters arrived to eat you. "
102 . "Maybe you can find comfort in the thought that your body was quite satisfying in taste... "
103 . "H<You disconnected too long without having used a savebed.>",
104 cf::NDI_RED
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
128 );
129 }
62 } 130 }
63}
64 131
65# delete a player directory, be non-blocking AND synchronous... 132 $ob->goto ($map, $x, $y);
66# (thats hard, so we crap out and fork). 133}
134
135sub encode_password($) {
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 $pass eq pack "H*", $1;
147 } else {
148 return $token eq crypt $pass, $token;
149 }
150}
151
152# delete a player directory
67sub nuke_playerdir { 153sub nuke_playerdir {
68 my ($user) = @_; 154 my ($user) = @_;
69 155
156 my $lock = cf::lock_acquire "ext::login::nuke_playerdir";
157
158 my $temp = "$PLAYERDIR/~$Coro::current~deleting~";
70 aio_stat "$PLAYERDIR/$user"; 159 aio_rename "$PLAYERDIR/$user", $temp;
71 system "cd \Q$PLAYERDIR\E " 160 IO::AIO::aio_rmtree $temp;
72 . "&& mv \Q$user\E ~\Q$Coro::current\E~deleting~ 2>/dev/null "
73 . "&& (rm -rf ~\Q$Coro::current\E~deleting~ &)";
74} 161}
75 162
76sub addme { 163cf::client->attach (on_addme => sub {
77 my ($ns) = @_; 164 my ($ns) = @_;
78 165
79 $ns->destroy if $ns->pl; 166 $ns->pl and return $ns->destroy;
80 167
81 $ns->async (sub { 168 $ns->async (sub {
169 $Coro::current->{desc} = "addme init";
170
82 my ($user, $pass); 171 my ($user, $pass);
83 172
84 $ns->send_packet ("addme_success"); 173 $ns->send_packet ("addme_success");
85 174
86 for (;;) { 175 for (;;) {
… …
91 cf::NDI_BLUE 180 cf::NDI_BLUE
92 ); 181 );
93 182
94 # read username 183 # read username
95 while () { 184 while () {
96 $user = query $ns, 0, "What is your name?\n:"; 185 $user = query $ns, 0, "What is your name? (login names are case-sensitive)\n:";
97 186
98 if ($cf::LOGIN_LOCK{$user}) { 187 if ($cf::LOGIN_LOCK{$user}) {
99 $ns->send_drawinfo ( 188 $ns->send_drawinfo (
100 "That username is currently used in another login session. " 189 "That username is currently used in another login session. "
101 . "Chose another, or wait till the other session has ended.", 190 . "Chose another, or wait till the other session has ended.",
102 cf::NDI_RED 191 cf::NDI_RED
103 ); 192 );
104 } elsif ($user =~ /^[a-zA-Z0-9][a-zA-Z0-9\-_]{2,17}$/) { 193 } elsif ($user =~ /^[a-zA-Z0-9][a-zA-Z0-9\-_]{2,19}\z/) {
105 last; 194 last;
106 } else { 195 } else {
107 $ns->send_drawinfo ( 196 $ns->send_drawinfo (
108 "Your username contains illegal characters " 197 "Your username contains illegal characters "
109 . "(only a-z, A-Z and 0-9 are allowed), " 198 . "(only a-z, A-Z and 0-9 are allowed), "
110 . "or is not between 3 and 18 characters in length.", 199 . "or is not between 3 and 20 characters in length.",
111 cf::NDI_RED 200 cf::NDI_RED
112 ); 201 );
113 } 202 }
203 Coro::Timer::sleep 0.4;
114 } 204 }
115 205
116 check_playing $ns, $user and next; 206 check_playing $ns, $user and next;
207
208 $Coro::current->{desc} = "addme($user) pass";
117 209
118 $ns->send_drawinfo ( 210 $ns->send_drawinfo (
119 "Welcome $user, please enter your password now. " 211 "Welcome $user, please enter your password now. "
120 . "New users should now choose a password. " 212 . "New users should now choose a password. "
121 . "Anything your client lets you enter is fine.", 213 . "Anything your client lets you enter is fine.",
… …
129 $ns->send_drawinfo ( 221 $ns->send_drawinfo (
130 "Try to use at least three characters as your password please, " 222 "Try to use at least three characters as your password please, "
131 . "that cannot be too much to ask for :)", 223 . "that cannot be too much to ask for :)",
132 cf::NDI_RED 224 cf::NDI_RED
133 ); 225 );
226 Coro::Timer::sleep 0.4;
134 } 227 }
135 228
136 # lock this username for the remainder of this login session 229 # lock this username for the remainder of this login session
137 if ($cf::LOGIN_LOCK{$user}) { 230 if ($cf::LOGIN_LOCK{$user}) {
138 $ns->send_drawinfo ( 231 $ns->send_drawinfo (
… …
144 } 237 }
145 local $cf::LOGIN_LOCK{$user} = 1; 238 local $cf::LOGIN_LOCK{$user} = 1;
146 239
147 check_playing $ns, $user and next; 240 check_playing $ns, $user and next;
148 241
242 $Coro::current->{desc} = "addme($user) check";
243
149 # try to read the user file and check the password 244 # try to read the user file and check the password
150 if (my $fh = aio_open cf::player::path $user, O_RDONLY, 0) { 245 if (my $pl = cf::player::find $user) {
246 aio_stat $pl->path and next;
151 my $mtime = (stat $fh)[9]; 247 my $mtime = (stat _)[9];
248 my $token = $pl->password;
152 249
153 0 < aio_read $fh, 0, 16384, my $buf, 0 or next; 250 if ($cf::CFG{ext_login_nocheck} or compare_password $pass, $token) {
154 $buf =~ /^password (\S+)$/m or next; 251 $pl->password (encode_password $pass); # make sure we store the new encoding #d#
155 my $hash = $1;
156
157 if ($hash eq crypt $pass, $hash) {
158 nuke_str $pass; 252 nuke_str $pass;
159 # password matches, wonderful 253 # password matches, wonderful
160 my $pl = cf::player::find $user or next; 254 my $pl = cf::player::find $user or next;
161 $pl->connect ($ns); 255 $pl->connect ($ns);
162 check_clean_save $pl; 256 enter_map $pl;
163 $pl->{clean_save} = 1;
164 last; 257 last;
165 } elsif (can_cleanup $buf, $mtime) { 258 } elsif (can_cleanup $pl, $mtime) {
166 Coro::Timer::sleep 1; 259 Coro::Timer::sleep 1;
167 260
168 $ns->send_drawinfo ( 261 $ns->send_drawinfo (
169 "Player exists, but password does not match. If this is your account, " 262 "Player exists, but password does not match. If this is your account, "
170 . "please try again. If not, you can now decide to take over this account " 263 . "please try again. If not, you can now decide to take over this account "
… …
178 271
179 # check if the file hasn't changed 272 # check if the file hasn't changed
180 aio_stat cf::player::path $user and next; 273 aio_stat cf::player::path $user and next;
181 $mtime == (stat _)[9] or next; 274 $mtime == (stat _)[9] or next;
182 275
183 nuke_playerdir $user; 276 $pl->quit_character;
184 277
185 # fall through to creation 278 # fall through to creation
186 } else { 279 } else {
187 nuke_str $pass; 280 nuke_str $pass;
188 281
… …
193 . "(check for Numlock and other semi-obvious error sources).", 286 . "(check for Numlock and other semi-obvious error sources).",
194 cf::NDI_RED 287 cf::NDI_RED
195 ); 288 );
196 next; 289 next;
197 } 290 }
291 } else {
292 # unable to load the playerfile:
293 # check whether the player dir exists, which means the file is corrupted or
294 # something very similar.
295 if (!aio_stat cf::player::playerdir $user) {
296 $ns->send_drawinfo (
297 "Unable to retrieve this player. It might be a locked or broken account. "
298 . "If this is your account, ask a dungeon master for assistance. "
299 . "Otherwise choose a different login name.",
300 cf::NDI_RED
301 );
302 next;
303 }
198 } 304 }
199 305
200 # the rest of this function is character creation 306 # the rest of this function is character creation
307 $Coro::current->{desc} = "addme($user) chargen";
201 308
202 # just to make sure nothing is left over 309 # just to make sure nothing is left over
310 # normally, nothing is there.
203 nuke_playerdir $user; 311 nuke_playerdir $user;
204 312
205 my $pass2 = query $ns, cf::CS_QUERY_HIDEINPUT, "Please type your password again."; 313 my $pass2 = query $ns, cf::CS_QUERY_HIDEINPUT, "Please type your password again.";
206 314
207 if ($pass2 ne $pass) { 315 if ($pass2 ne $pass) {
… …
209 nuke_str $pass2; 317 nuke_str $pass2;
210 $ns->send_drawinfo ( 318 $ns->send_drawinfo (
211 "The passwords do not match, please try again.", 319 "The passwords do not match, please try again.",
212 cf::NDI_RED 320 cf::NDI_RED
213 ); 321 );
322 Coro::Timer::sleep 0.5;
214 next; 323 next;
215 } 324 }
216 325
217 nuke_str $pass2; 326 nuke_str $pass2;
218 327
219 my $pl = cf::player::new $user; 328 my $pl = cf::player::new $user;
220 $pl->password (crypt $pass, join '', ('.', '/', 0..9, 'A'..'Z', 'a'..'z')[rand 64, rand 64]); 329 $pl->password (encode_password $pass);
221 nuke_str $pass; 330 nuke_str $pass;
222 $pl->connect ($ns); 331 $pl->connect ($ns);
223
224 my $ob = $pl->ob; 332 my $ob = $pl->ob;
333
334 $ob->goto ($pl->maplevel, $ob->x, $ob->y);
225 335
226 while () { 336 while () {
227 $ob->update_stats; 337 $ob->update_stats;
228 $pl->save_stats; 338 $pl->save_stats;
229 339
… …
239 $ob->swap_stats ($res - 1, $swap - 1); 349 $ob->swap_stats ($res - 1, $swap - 1);
240 } 350 }
241 } else { 351 } else {
242 $ob->roll_stats; 352 $ob->roll_stats;
243 } 353 }
354
355 Coro::Timer::sleep 0.05;
244 } 356 }
245 357
246 $ob->set_animation (2); 358 $ob->set_animation (2);
247 $ob->add_statbonus; 359 $ob->add_statbonus;
248 360
249 $ns->send_drawinfo ($ob->msg, cf::NDI_BLUE); 361 while () {
250 $ns->send_packet (sprintf "query %d %s", cf::CS_QUERY_SINGLECHAR, 362 $ns->send_msg ("chargen-race-title", ucfirst $pl->title, -1);
363 my $msg = $ob->msg;
364 $msg =~ s/(?<=\S)\n(?=\S)/ /g;
365 $ns->send_msg ("chargen-race-description", $msg, cf::NDI_BLUE);
366
367 my $res = query $ns, cf::CS_QUERY_SINGLECHAR,
251 "Now choose a character.\nPress any key to change outlook.\nPress `d' when you're pleased.\n"); 368 "Now choose a character.\nPress any key to change outlook.\nPress `d' when you're pleased.\n";
252 369
253 $ns->state (cf::ST_CHANGE_CLASS); 370 last if $res =~ /[dD]/;
371
372 $pl->chargen_race_next;
373 Coro::Timer::sleep 0.05;
374 }
375
376 # create the playerdir, if necessary, as chargen_race_done did it before
377 # presumably because of unique maps
378 aio_mkdir playerdir $pl, 0770;
379 $pl->chargen_race_done;
380
381 while () {
382 my $res = query $ns, cf::CS_QUERY_SINGLECHAR,
383 "Now choose a gender.\nPress 'f' to become female, and 'm' to become male.\n";
384
385 if ($res =~ /^[fF]/) {
386 $pl->gender (1);
387 last;
388 } elsif ($res =~ /^[mM]/) {
389 $pl->gender (0);
390 last;
391 }
392 Coro::Timer::sleep 0.05;
393 }
394
395 $ob->reply (undef, "Welcome to Deliantra!");
396
397 # XXX: Workaround for delayed client ext protocol handshake
398 $pl->esrv_new_player;
399
254 delete $pl->{deny_save};#d# too early 400 delete $pl->{deny_save};
255 401
256 last; 402 last;
257 } 403 }
404
405 if (0 < Coro::AIO::aio_load "$cf::CONFDIR/motd", my $motd) {
406 $ns->send_msg ("c/motd" => $motd, cf::NDI_CLEAR);
407 }
258 }); 408 });
259} 409});
410
411cf::register_command password => sub {
412 my ($pl, $arg) = @_;
413
414 unless ($pl->flag (cf::FLAG_WIZ)) {
415 $pl->message (
416 "The password can currently only changed by a DM.",
417 cf::NDI_UNIQUE | cf::NDI_REPLY);
418 return;
419 }
420
421 my (@args) = split /\s+/, $arg;
422 my ($player, $new_pw) = @args;
423
424 if ($pl->flag (cf::FLAG_WIZ) && $player eq '') {
425 $pl->message (
426 "Usage: password <player> [<new password>]",
427 cf::NDI_UNIQUE | cf::NDI_REPLY);
428 return;
429 }
430
431 if ($new_pw eq '') {
432 $new_pw =
433 join '',
434 map { ('.', '/', 0..9, 'A'..'Z', 'a'..'z')[(cf::rndm 64)] }
435 1..9;
436 }
437
438 cf::async {
439 my $plc = cf::player::find $player;
440 if ($plc) {
441 $plc->password (encode_password $new_pw);
442 $pl->message (
443 "Ok, changed password of '$player' to '$new_pw'!",
444 cf::NDI_UNIQUE | cf::NDI_RED | cf::NDI_REPLY);
445 } else {
446 $pl->message (
447 "Fail! Couldn't set password for '$player', "
448 . "he doesn't seem to exist!",
449 cf::NDI_UNIQUE | cf::NDI_RED | cf::NDI_REPLY);
450 }
451 };
452};
260 453
261cf::register_command quit => sub { 454cf::register_command quit => sub {
262 my ($ob, $arg) = @_; 455 my ($ob, $arg) = @_;
263 456
264 $ob->reply (undef, 457 $ob->send_msg (undef,
265 "Quitting will delete your character PERMANENTLY: It will be gone forever and any progress will be lost. " 458 "Quitting will delete your character PERMANENTLY: It will be gone forever and any progress will be lost. "
266 . "If you are sure you want to do this, then use the quit_character command instead of quit.", 459 . "If you are sure you want to do this, then use the quit_character command instead of quit.",
267 cf::NDI_UNIQUE | cf::NDI_RED); 460 cf::NDI_UNIQUE | cf::NDI_RED | cf::NDI_REPLY);
268}; 461};
269 462
270cf::register_command quit_character => sub { 463cf::register_command quit_character => sub {
271 my ($ob, $arg) = @_; 464 my ($ob, $arg) = @_;
272 465
273 my $pl = $ob->contr; 466 my $pl = $ob->contr;
274 467
275 $pl->ns->query (cf::CS_QUERY_SINGLECHAR, "Do you want to PERMANENTLY delete your character and all associated data (y/n)?", sub { 468 $pl->ns->query (cf::CS_QUERY_SINGLECHAR, "Do you want to PERMANENTLY delete your character and all associated data (y/n)?", sub {
276 if ($_[0] !~ /^[yY]/) { 469 if ($_[0] !~ /^[yY]/) {
277 $ob->reply (undef, 470 $ob->send_msg (undef, "Ok, not not quitting then.", cf::NDI_UNIQUE | cf::NDI_RED | cf::NDI_REPLY);
278 "Ok, not not quitting then.",
279 cf::NDI_UNIQUE | cf::NDI_RED);
280 } else { 471 } else {
281 $ob->reply (undef, 472 $ob->send_msg (undef, "Ok, quitting, hope to see you again.", cf::NDI_UNIQUE | cf::NDI_RED | cf::NDI_REPLY);
282 "Ok, quitting, hope to see you again.", 473 cf::async {
283 cf::NDI_UNIQUE | cf::NDI_RED);
284 $pl->ns->flush;
285 $pl->quit_character; 474 $pl->quit_character;
475 };
286 } 476 }
287 }); 477 });
288}; 478};
289 479
290cf::object->attach ( 480cf::object->attach (
… …
297 my $pl = $ob->contr; 487 my $pl = $ob->contr;
298 488
299 # update respawn position 489 # update respawn position
300 $pl->savebed ($bed->map->path, $bed->x, $bed->y); 490 $pl->savebed ($bed->map->path, $bed->x, $bed->y);
301 491
302 $pl->killer ("left"); 492 cf::async {
303 $ob->check_score; 493 my $killer = cf::arch::get "killer_logout"; $pl->killer ($killer); $killer->destroy;
494 ext::highscore::check $ob;
304 495
305 $ob->reply (undef, "In the future, you will wake up here when you die."); 496 $pl->save;
306 497
498 $ob->send_msg ($cf::SAY_CHANNEL => "In the future, you will wake up here when you die.", cf::NDI_DEF | cf::NDI_REPLY);
499
307 $pl->ns->query (cf::CS_QUERY_SINGLECHAR, "Do you want to continue playing (y/n)?", sub { 500 $pl->ns->query (cf::CS_QUERY_SINGLECHAR, "Do you want to continue playing (y/n)?", sub {
308 if ($_[0] !~ /^[yY]/) { 501 if ($_[0] !~ /^[yY]/) {
309 $pl->invoke (cf::EVENT_PLAYER_LOGOUT, 1); 502 $pl->invoke (cf::EVENT_PLAYER_LOGOUT, 1);
310 $pl->deactivate; 503 $pl->deactivate;
311 $pl->ns->destroy; 504 $pl->ns->destroy;
312 } else { 505 }
313 cf::async { $pl->save };
314 } 506 });
315 }); 507 };
316 }, 508 },
317); 509);
318 510
319cf::player->attach ( 511cf::player->attach (
320 on_login => sub { 512 on_login => sub {
… …
329 521
330 if ($cleanly) { 522 if ($cleanly) {
331 $_->ob->message ("$name left the game.", cf::NDI_DK_ORANGE | cf::NDI_UNIQUE) for cf::player::list; 523 $_->ob->message ("$name left the game.", cf::NDI_DK_ORANGE | cf::NDI_UNIQUE) for cf::player::list;
332 } else { 524 } else {
333 $_->ob->message ("$name uncerimoniously disconnected.", cf::NDI_DK_ORANGE | cf::NDI_UNIQUE) for cf::player::list; 525 $_->ob->message ("$name uncerimoniously disconnected.", cf::NDI_DK_ORANGE | cf::NDI_UNIQUE) for cf::player::list;
334 delete $pl->{clean_save}; 526 $pl->{unclean_save} = $cf::RUNTIME
527 unless safe_spot $pl;
335 } 528 }
336 }, 529 },
337); 530);
338 531
339cf::client->attach (
340 on_addme => \&addme,
341);
342
343#############################################################################
344
345our $SCHEDULE_INTERVAL = 10; # time the player scheduler sleeps between runs
346our $SAVE_TIMEOUT = 200; # save players every n seconds
347our $SAVE_INTERVAL = 0.1; # save at max. one player every $SAVE_INTERVAL
348
349our $SCHEDULER = cf::async_ext {
350 while () {
351 Coro::Timer::sleep $SCHEDULE_INTERVAL;
352
353 # this weird form of iteration over values is used because
354 # the hash changes underneath us frequently, and for
355 # keeps a direct reference to the value without (in 5.8 perls)
356 # keeping a reference, so this is prone to crashes or worse.
357 my @players = keys %cf::PLAYER;
358 for (@players) {
359 my $pl = $cf::PLAYER{$_}
360 or next;
361 $pl->valid or next;
362
363 eval {
364 if ($pl->{last_save} + $SAVE_TIMEOUT <= $cf::RUNTIME) {
365 $pl->save;
366 Coro::Timer::sleep $SAVE_INTERVAL;
367 }
368
369 unless ($pl->active) {
370 # check refcounts, this is tricky and needs to be adjusted to fit server internals
371 my $ob = $pl->ob;
372 Scalar::Util::weaken $pl;
373 Scalar::Util::weaken $ob;
374 my $a_ = $pl->refcnt;
375 my $b_ = $ob->refcnt;
376 my $pl_ref = $pl->refcnt_cnt;
377 my $ob_ref = $ob->refcnt_cnt;
378
379 if ($pl_ref == 2 && $ob_ref == 1) {
380 warn "player-scheduler destroy ", $ob->name;#d#
381 delete $cf::PLAYER{$ob->name};
382 # pl_ref == one from object + one from cf::PLAYER
383 # ob_ref == one from simply being an object
384 $ob->destroy;
385 $pl->destroy;
386 } else {
387 warn "player-scheduler refcnt ", $ob->name, " $pl_ref,$a_ $ob_ref,$b_\n";#d#
388 }
389 }
390 };
391 warn $@ if $@;
392 Coro::cede;
393 };
394 }
395};
396
397$SCHEDULER->prio (1);
398

Diff Legend

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