| 1 | #! perl |
1 | #! perl # MANDATORY |
| 2 | |
2 | |
| 3 | # login handling |
3 | # login handling |
| 4 | |
4 | |
| 5 | use Fcntl; |
5 | use Fcntl; |
| 6 | use Coro::AIO; |
6 | use Coro::AIO; |
|
|
7 | use List::Util qw(min max); |
| 7 | |
8 | |
| 8 | my $PLAYERDIR = sprintf "%s/%s", cf::localdir, cf::playerdir; |
9 | my $PLAYERDIR = sprintf "%s/%s", cf::localdir, cf::playerdir; |
| 9 | |
10 | |
| 10 | # paranoia function to overwrite a string-in-place |
11 | # paranoia function to overwrite a string-in-place |
| 11 | sub nuke_str { |
12 | sub nuke_str { |
| … | |
… | |
| 14 | |
15 | |
| 15 | sub query { |
16 | sub query { |
| 16 | my ($ns, $flags, $text) = @_; |
17 | my ($ns, $flags, $text) = @_; |
| 17 | |
18 | |
| 18 | my $current = $Coro::current; |
19 | my $current = $Coro::current; |
| 19 | $ns->query ($flags, $text, sub { $current->ready; $current = $_[0]; }); |
20 | $ns->query ($flags, $text, sub { $current->ready; $current = $_[0] }); |
| 20 | Coro::schedule while ref $current; |
21 | Coro::schedule while ref $current; |
| 21 | |
22 | |
| 22 | $current |
23 | $current |
| 23 | } |
24 | } |
| 24 | |
25 | |
| 25 | sub can_cleanup { |
26 | sub can_cleanup { |
| 26 | my ($playerfile, $mtime) = @_; |
27 | my ($pl, $mtime) = @_; |
| 27 | |
28 | |
| 28 | my $age = time - $mtime; |
29 | my $age = time - $mtime; |
| 29 | my $level = $playerfile =~ /^level (\d+)$/m ? $1 : return; |
30 | my $level = $pl->ob->level; |
| 30 | |
31 | |
| 31 | ($level <= 3 && $age > 7 * 86400) # 7 days for level 0..3 |
32 | ($level <= 3 && $age > 7 * 86400) # 7 days for level 0..3 |
| 32 | || ($level <= 9 && $age > 90 * 86400) # 3 months for level 4..9 |
33 | || ($level <= 9 && $age > 90 * 86400) # 3 months for level 4..9 |
| 33 | || ($level <= 20 && $age > 180 * 86400) # 6 months for level 10..20 |
34 | || ($level <= 20 && $age > 180 * 86400) # 6 months for level 10..20 |
| 34 | || $age > 700 * 86400 # 2 years for everybody else |
35 | || $age > 700 * 86400 # 2 years for everybody else |
| … | |
… | |
| 55 | } |
56 | } |
| 56 | |
57 | |
| 57 | sub check_clean_save { |
58 | sub check_clean_save { |
| 58 | my ($pl) = @_; |
59 | my ($pl) = @_; |
| 59 | |
60 | |
| 60 | unless (delete $pl->{clean_save}) { |
61 | if (my $time = delete $pl->{unclean_save}) { |
|
|
62 | $pl->ns->send_drawinfo ( |
|
|
63 | "You didn't use a savebed to leave this realm. This is very dangerous, " |
|
|
64 | . "as lots of things could happen when you leave by other means, such as cave-ins, " |
|
|
65 | . "or monsters suddenly snapping your body. Better use a savebed next time.", |
|
|
66 | cf::NDI_RED |
|
|
67 | ); |
| 61 | #d#TODO |
68 | #d#TODO |
| 62 | } |
69 | } |
| 63 | } |
70 | } |
| 64 | |
71 | |
| 65 | # delete a player directory, be non-blocking AND synchronous... |
72 | # delete a player directory, be non-blocking AND synchronous... |
| … | |
… | |
| 71 | system "cd \Q$PLAYERDIR\E " |
78 | system "cd \Q$PLAYERDIR\E " |
| 72 | . "&& mv \Q$user\E ~\Q$Coro::current\E~deleting~ 2>/dev/null " |
79 | . "&& mv \Q$user\E ~\Q$Coro::current\E~deleting~ 2>/dev/null " |
| 73 | . "&& (rm -rf ~\Q$Coro::current\E~deleting~ &)"; |
80 | . "&& (rm -rf ~\Q$Coro::current\E~deleting~ &)"; |
| 74 | } |
81 | } |
| 75 | |
82 | |
|
|
83 | sub send_capabilities { |
|
|
84 | my ($ns) = @_; |
|
|
85 | |
|
|
86 | return unless $ns->extcmd; |
|
|
87 | |
|
|
88 | $ns->ext_event (capabilities => |
|
|
89 | # id, name, flags (1 == 2d), edge length |
|
|
90 | tileset => [[1, "default 64x64 faceset", 1, 64], [0, "default 32x32 faceset", 1, 32]], |
|
|
91 | ); |
|
|
92 | } |
|
|
93 | |
|
|
94 | sub setup { |
|
|
95 | my ($ns, $args) = @_; |
|
|
96 | |
|
|
97 | # run through the cmds of setup |
|
|
98 | # syntax is setup <cmdname1> <parameter> <cmdname2> <parameter> ... |
|
|
99 | # |
|
|
100 | # we send the status of the cmd back, or a FALSE is the cmd is the server unknown |
|
|
101 | # The client then must sort this out |
|
|
102 | |
|
|
103 | my %setup = split / +/, $args; |
|
|
104 | while (my ($k, $v) = each %setup) { |
|
|
105 | if ($k eq "sound") { |
|
|
106 | $ns->sound ($v); |
|
|
107 | |
|
|
108 | } elsif ($k eq "exp64") { |
|
|
109 | $setup{$k} = 1; |
|
|
110 | |
|
|
111 | } elsif ($k eq "spellmon") { |
|
|
112 | $ns->monitor_spells ($v); |
|
|
113 | |
|
|
114 | } elsif ($k eq "darkness") { |
|
|
115 | $ns->darkness ($v); |
|
|
116 | |
|
|
117 | } elsif ($k eq "map1cmd") { |
|
|
118 | $ns->mapmode (cf::Map1Cmd) if $v > 0; |
|
|
119 | |
|
|
120 | } elsif ($k eq "map1acmd") { |
|
|
121 | $ns->mapmode (cf::Map1aCmd) if $v > 0; |
|
|
122 | |
|
|
123 | } elsif ($k eq "map2cmd") { |
|
|
124 | # gcfclient bug, map1acmd is sent too late |
|
|
125 | $ns->mapmode (cf::Map1aCmd); |
|
|
126 | $setup{$k} = "FALSE"; |
|
|
127 | |
|
|
128 | } elsif ($k eq "newmapcmd") { |
|
|
129 | $ns->newmapcmd ($v); |
|
|
130 | |
|
|
131 | } elsif ($k eq "mapinfocmd") { |
|
|
132 | $ns->mapinfocmd ($v); |
|
|
133 | |
|
|
134 | } elsif ($k eq "extcmd") { |
|
|
135 | $ns->extcmd ($v > 0); |
|
|
136 | send_capabilities $ns; |
|
|
137 | |
|
|
138 | } elsif ($k eq "extmap") { |
|
|
139 | $ns->extmap ($v); |
|
|
140 | |
|
|
141 | } elsif ($k eq "facecache") { |
|
|
142 | if (!$v) { |
|
|
143 | $v = 1; |
|
|
144 | $setup{$k} = $v; |
|
|
145 | $ns->send_drawinfo ("(trying to forcefully enable facecaching)", cf::NDI_RED); |
|
|
146 | } |
|
|
147 | |
|
|
148 | $ns->facecache ($v); |
|
|
149 | |
|
|
150 | } elsif ($k eq "faceset") { |
|
|
151 | $ns->faceset (0); |
|
|
152 | $setup{$k} = 0; |
|
|
153 | # $ns->image2 (1) |
|
|
154 | |
|
|
155 | } elsif ($k eq "tileset") { |
|
|
156 | $setup{$k} = $ns->faceset ($v & 1); |
|
|
157 | |
|
|
158 | } elsif ($k eq "itemcmd") { |
|
|
159 | # Version of the item protocol command to use. Currently, |
|
|
160 | # only supported versions are 1 and 2. Using a numeric |
|
|
161 | # value will make it very easy to extend this in the future. |
|
|
162 | $ns->itemcmd ($v) if $v >= 1 && $v <= 2; |
|
|
163 | |
|
|
164 | $setup{$k} = $ns->itemcmd; |
|
|
165 | |
|
|
166 | } elsif ($k eq "mapsize") { |
|
|
167 | my ($x, $y) = split /x/, $v; |
|
|
168 | |
|
|
169 | $ns->mapx ($x = max 9, min cf::MAP_CLIENT_X, ($x - 1) | 1); |
|
|
170 | $ns->mapy ($y = max 9, min cf::MAP_CLIENT_Y, ($y - 1) | 1); |
|
|
171 | |
|
|
172 | $setup{$k} = "${x}x${y}"; |
|
|
173 | |
|
|
174 | } elsif ($k eq "extendedMapInfos") { |
|
|
175 | $ns->ext_mapinfos ($v); |
|
|
176 | |
|
|
177 | } elsif ($k eq "extendedTextInfos") { |
|
|
178 | $ns->has_readable_type ($v); |
|
|
179 | |
|
|
180 | } elsif ($k eq "smoothing") { # cfplus-style smoothing |
|
|
181 | $ns->smoothing ($v); |
|
|
182 | |
|
|
183 | } elsif ($k eq "fxix") { |
|
|
184 | $ns->fxix ($v); |
|
|
185 | |
|
|
186 | } else { |
|
|
187 | # other commands: |
|
|
188 | # sexp: no idea, probably for oudated servers |
|
|
189 | # tick: more stupidity, server should sned a tick per tick |
|
|
190 | |
|
|
191 | $setup{$k} = "FALSE"; |
|
|
192 | } |
|
|
193 | } |
|
|
194 | |
|
|
195 | $ns->send_packet (join " ", setup => %setup); |
|
|
196 | |
|
|
197 | cf::datalog setup => |
|
|
198 | request => $args, |
|
|
199 | reply => \%setup, |
|
|
200 | client => $ns->version, |
|
|
201 | ; |
|
|
202 | } |
|
|
203 | |
| 76 | sub addme { |
204 | sub addme { |
| 77 | my ($ns) = @_; |
205 | my ($ns) = @_; |
| 78 | |
206 | |
| 79 | $ns->destroy if $ns->pl; |
207 | if (!$ns->facecache) |
|
|
208 | { |
|
|
209 | $ns->send_drawinfo (<<EOF, cf::NDI_RED); |
|
|
210 | |
|
|
211 | |
|
|
212 | *** |
|
|
213 | *** WARNING: |
|
|
214 | *** Your client does not support face/image caching, |
|
|
215 | *** or it has been disabled. Face caching is mandatory |
|
|
216 | *** so please enable it or use a newer client. |
|
|
217 | *** |
|
|
218 | *** Look at your client preferences: |
|
|
219 | *** |
|
|
220 | *** CFPlus: all known versions automatically enable the facecache. |
|
|
221 | *** cfclient: use the -cache commandline option. |
|
|
222 | *** cfclient: map will not redraw automatically (bug). |
|
|
223 | *** gcfclient: use -cache commandline option, or enable |
|
|
224 | *** gcfclient: Client=>Configure=>Map & Image=>Cache Images. |
|
|
225 | *** jcrossclient: your client is broken, use CFPlus or gcfclient. |
|
|
226 | *** |
|
|
227 | *** |
|
|
228 | EOF |
|
|
229 | if ($ns->version =~ /jcrossclient/) { |
|
|
230 | # let them, for now |
|
|
231 | } else { |
|
|
232 | $ns->flush; |
|
|
233 | return $ns->destroy; |
|
|
234 | } |
|
|
235 | |
|
|
236 | # $ns->facecache = true; |
|
|
237 | } |
|
|
238 | |
|
|
239 | if ($ns->mapmode < cf::Map1aCmd) { |
|
|
240 | $ns->send_drawinfo (<<EOF, cf::NDI_RED); |
|
|
241 | |
|
|
242 | |
|
|
243 | *** |
|
|
244 | *** WARNING: |
|
|
245 | *** Your client is too old. Please upgrade to a newer version. |
|
|
246 | EOF |
|
|
247 | |
|
|
248 | $ns->flush; |
|
|
249 | return $ns->destroy; |
|
|
250 | } |
|
|
251 | |
|
|
252 | $ns->pl and return $ns->destroy; |
| 80 | |
253 | |
| 81 | $ns->async (sub { |
254 | $ns->async (sub { |
| 82 | my ($user, $pass); |
255 | my ($user, $pass); |
| 83 | |
256 | |
| 84 | $ns->send_packet ("addme_success"); |
257 | $ns->send_packet ("addme_success"); |
| … | |
… | |
| 99 | $ns->send_drawinfo ( |
272 | $ns->send_drawinfo ( |
| 100 | "That username is currently used in another login session. " |
273 | "That username is currently used in another login session. " |
| 101 | . "Chose another, or wait till the other session has ended.", |
274 | . "Chose another, or wait till the other session has ended.", |
| 102 | cf::NDI_RED |
275 | cf::NDI_RED |
| 103 | ); |
276 | ); |
| 104 | } elsif ($user =~ /^[a-zA-Z0-9][a-zA-Z0-9\-_]{2,17}$/) { |
277 | } elsif ($user =~ /^[a-zA-Z0-9][a-zA-Z0-9\-_]{2,17}\z/) { |
| 105 | last; |
278 | last; |
| 106 | } else { |
279 | } else { |
| 107 | $ns->send_drawinfo ( |
280 | $ns->send_drawinfo ( |
| 108 | "Your username contains illegal characters " |
281 | "Your username contains illegal characters " |
| 109 | . "(only a-z, A-Z and 0-9 are allowed), " |
282 | . "(only a-z, A-Z and 0-9 are allowed), " |
| … | |
… | |
| 145 | local $cf::LOGIN_LOCK{$user} = 1; |
318 | local $cf::LOGIN_LOCK{$user} = 1; |
| 146 | |
319 | |
| 147 | check_playing $ns, $user and next; |
320 | check_playing $ns, $user and next; |
| 148 | |
321 | |
| 149 | # try to read the user file and check the password |
322 | # try to read the user file and check the password |
| 150 | if (my $fh = aio_open cf::player::path $user, O_RDONLY, 0) { |
323 | if (my $pl = cf::player::find $user) { |
|
|
324 | aio_stat $pl->path and next; |
| 151 | my $mtime = (stat $fh)[9]; |
325 | my $mtime = (stat _)[9]; |
| 152 | |
|
|
| 153 | 0 < aio_read $fh, 0, 16384, my $buf, 0 or next; |
|
|
| 154 | $buf =~ /^password (\S+)$/m or next; |
|
|
| 155 | my $hash = $1; |
326 | my $hash = $pl->password; |
| 156 | |
327 | |
| 157 | if ($hash eq crypt $pass, $hash) { |
328 | if ($cf::CFG{ext_login_nocheck} or $hash eq crypt $pass, $hash) { |
| 158 | nuke_str $pass; |
329 | nuke_str $pass; |
| 159 | # password matches, wonderful |
330 | # password matches, wonderful |
| 160 | my $pl = cf::player::find $user or next; |
331 | my $pl = cf::player::find $user or next; |
| 161 | $pl->connect ($ns); |
332 | $pl->connect ($ns); |
| 162 | check_clean_save $pl; |
333 | check_clean_save $pl; |
| 163 | last; |
334 | last; |
| 164 | } elsif (can_cleanup $buf, $mtime) { |
335 | } elsif (can_cleanup $pl, $mtime) { |
| 165 | Coro::Timer::sleep 1; |
336 | Coro::Timer::sleep 1; |
| 166 | |
337 | |
| 167 | $ns->send_drawinfo ( |
338 | $ns->send_drawinfo ( |
| 168 | "Player exists, but password does not match. If this is your account, " |
339 | "Player exists, but password does not match. If this is your account, " |
| 169 | . "please try again. If not, you can now decide to take over this account " |
340 | . "please try again. If not, you can now decide to take over this account " |
| … | |
… | |
| 177 | |
348 | |
| 178 | # check if the file hasn't changed |
349 | # check if the file hasn't changed |
| 179 | aio_stat cf::player::path $user and next; |
350 | aio_stat cf::player::path $user and next; |
| 180 | $mtime == (stat _)[9] or next; |
351 | $mtime == (stat _)[9] or next; |
| 181 | |
352 | |
| 182 | nuke_playerdir $user; |
353 | $pl->quit_character; |
| 183 | |
354 | |
| 184 | # fall through to creation |
355 | # fall through to creation |
| 185 | } else { |
356 | } else { |
| 186 | nuke_str $pass; |
357 | nuke_str $pass; |
| 187 | |
358 | |
| 188 | Coro::Timer::sleep 1; |
359 | Coro::Timer::sleep 1; |
| 189 | |
360 | |
| 190 | $ns->send_drawinfo ( |
361 | $ns->send_drawinfo ( |
| 191 | "Wrong username or password. Please try again " |
362 | "Wrong username or password. Please try again " |
| 192 | . "(check for Numlock and other semi-obvious error sources).", |
363 | . "(check for Numlock and other semi-obvious error sources).", |
|
|
364 | cf::NDI_RED |
|
|
365 | ); |
|
|
366 | next; |
|
|
367 | } |
|
|
368 | } else { |
|
|
369 | # unable to load the playerfile: |
|
|
370 | # check wether the player dir exists, which means the file is corrupted or |
|
|
371 | # something very similar. |
|
|
372 | if (!aio_stat cf::player::playerdir $user) { |
|
|
373 | $ns->send_drawinfo ( |
|
|
374 | "Unable to retrieve this player. It might be a locked or broken account. " |
|
|
375 | . "If this is your account, ask a dungeon master for assistance. " |
|
|
376 | . "Otherwise choose a different login name.", |
| 193 | cf::NDI_RED |
377 | cf::NDI_RED |
| 194 | ); |
378 | ); |
| 195 | next; |
379 | next; |
| 196 | } |
380 | } |
| 197 | } |
381 | } |
| … | |
… | |
| 255 | last; |
439 | last; |
| 256 | } |
440 | } |
| 257 | }); |
441 | }); |
| 258 | } |
442 | } |
| 259 | |
443 | |
| 260 | # "Quitting will delete your character PERMANENTLY. If you are sure you want to do this, then use the quit_character command instead of quit." |
444 | cf::register_command quit => sub { |
| 261 | # "Quitting will delete your character.\nAre you sure you want to quit (y/n):" |
445 | my ($ob, $arg) = @_; |
| 262 | # quit, quit_character, save |
446 | |
|
|
447 | $ob->reply (undef, |
|
|
448 | "Quitting will delete your character PERMANENTLY: It will be gone forever and any progress will be lost. " |
|
|
449 | . "If you are sure you want to do this, then use the quit_character command instead of quit.", |
|
|
450 | cf::NDI_UNIQUE | cf::NDI_RED); |
|
|
451 | }; |
|
|
452 | |
|
|
453 | cf::register_command quit_character => sub { |
|
|
454 | my ($ob, $arg) = @_; |
|
|
455 | |
|
|
456 | my $pl = $ob->contr; |
|
|
457 | |
|
|
458 | $pl->ns->query (cf::CS_QUERY_SINGLECHAR, "Do you want to PERMANENTLY delete your character and all associated data (y/n)?", sub { |
|
|
459 | if ($_[0] !~ /^[yY]/) { |
|
|
460 | $ob->reply (undef, |
|
|
461 | "Ok, not not quitting then.", |
|
|
462 | cf::NDI_UNIQUE | cf::NDI_RED); |
|
|
463 | } else { |
|
|
464 | $ob->reply (undef, |
|
|
465 | "Ok, quitting, hope to see you again.", |
|
|
466 | cf::NDI_UNIQUE | cf::NDI_RED); |
|
|
467 | $pl->ns->flush; |
|
|
468 | cf::async { $pl->quit_character }; |
|
|
469 | } |
|
|
470 | }); |
|
|
471 | }; |
| 263 | |
472 | |
| 264 | cf::object->attach ( |
473 | cf::object->attach ( |
| 265 | type => cf::SAVEBED, |
474 | type => cf::SAVEBED, |
| 266 | on_apply => sub { |
475 | on_apply => sub { |
| 267 | my ($bed, $ob) = @_; |
476 | my ($bed, $ob) = @_; |
| 268 | |
477 | |
| 269 | return cf::override 0 unless $ob->type == cf::PLAYER; |
478 | return cf::override 0 unless $ob->type == cf::PLAYER; |
| 270 | |
479 | |
| 271 | my $pl = $ob->pl; |
480 | my $pl = $ob->contr; |
| 272 | |
481 | |
| 273 | # update respawn position |
482 | # update respawn position |
| 274 | $pl->savebed ($bed->map->path, $bed->x, $bed->y); |
483 | $pl->savebed ($bed->map->path, $bed->x, $bed->y); |
|
|
484 | cf::async { $pl->save }; |
| 275 | |
485 | |
| 276 | $pl->killer ("left"); |
486 | $pl->killer ("left"); |
| 277 | $ob->check_score; |
487 | $ob->check_score; |
| 278 | |
488 | |
| 279 | $ob->reply (undef, "In the future, you will wake up here when you die."); |
489 | $ob->reply (undef, "In the future, you will wake up here when you die."); |
| … | |
… | |
| 282 | if ($_[0] !~ /^[yY]/) { |
492 | if ($_[0] !~ /^[yY]/) { |
| 283 | $pl->invoke (cf::EVENT_PLAYER_LOGOUT, 1); |
493 | $pl->invoke (cf::EVENT_PLAYER_LOGOUT, 1); |
| 284 | $pl->deactivate; |
494 | $pl->deactivate; |
| 285 | $pl->ns->destroy; |
495 | $pl->ns->destroy; |
| 286 | } else { |
496 | } else { |
| 287 | cf::async { |
497 | cf::async { $pl->save }; |
| 288 | $pl->{clean_save} = 1; |
|
|
| 289 | $pl->save; |
|
|
| 290 | }; |
|
|
| 291 | } |
498 | } |
| 292 | }); |
499 | }); |
| 293 | }, |
500 | }, |
| 294 | ); |
501 | ); |
| 295 | |
502 | |
| … | |
… | |
| 306 | |
513 | |
| 307 | if ($cleanly) { |
514 | if ($cleanly) { |
| 308 | $_->ob->message ("$name left the game.", cf::NDI_DK_ORANGE | cf::NDI_UNIQUE) for cf::player::list; |
515 | $_->ob->message ("$name left the game.", cf::NDI_DK_ORANGE | cf::NDI_UNIQUE) for cf::player::list; |
| 309 | } else { |
516 | } else { |
| 310 | $_->ob->message ("$name uncerimoniously disconnected.", cf::NDI_DK_ORANGE | cf::NDI_UNIQUE) for cf::player::list; |
517 | $_->ob->message ("$name uncerimoniously disconnected.", cf::NDI_DK_ORANGE | cf::NDI_UNIQUE) for cf::player::list; |
|
|
518 | $pl->{unclean_save} = $cf::RUNTIME; |
| 311 | } |
519 | } |
| 312 | }, |
520 | }, |
| 313 | ); |
521 | ); |
| 314 | |
522 | |
| 315 | cf::client->attach ( |
523 | cf::client->attach ( |
| 316 | on_addme => \&addme, |
524 | on_addme => \&addme, |
|
|
525 | on_setup => \&setup, |
| 317 | ); |
526 | ); |
| 318 | |
527 | |
| 319 | ############################################################################# |
528 | ############################################################################# |
| 320 | |
529 | |
| 321 | our $SCHEDULE_INTERVAL = 10; # time the player scheduler sleeps between runs |
530 | our $SCHEDULE_INTERVAL = 10; # time the player scheduler sleeps between runs |
| 322 | our $SWAP_TIMEOUT = 30; # time after which an unused player is evicted form memory |
|
|
| 323 | our $SAVE_TIMEOUT = 20; # save players every n seconds |
531 | our $SAVE_TIMEOUT = 20; # save players every n seconds |
| 324 | our $SAVE_INTERVAL = 0.1; # save at max. one player every $SAVE_INTERVAL |
|
|
| 325 | |
532 | |
| 326 | our $SCHEDULER = cf::async_ext { |
533 | our $SCHEDULER = cf::async_ext { |
|
|
534 | my $schedule_interval = Coro::Event->timer (after => 1, interval => $SCHEDULE_INTERVAL); |
| 327 | while () { |
535 | while () { |
| 328 | Coro::Timer::sleep $SCHEDULE_INTERVAL; |
536 | $schedule_interval->next; |
| 329 | |
537 | |
| 330 | # this weird form of iteration over values is used because |
538 | # this weird form of iteration over values is used because |
| 331 | # the hash changes underneath us frequently, and for |
539 | # the hash changes underneath us frequently, and for |
| 332 | # keeps a direct reference to the value without (in 5.8 perls) |
540 | # keeps a direct reference to the value without (in 5.8 perls) |
| 333 | # keeping a reference, so this is prone to crashes or worse. |
541 | # keeping a reference, so this is prone to crashes or worse. |
| … | |
… | |
| 337 | or next; |
545 | or next; |
| 338 | $pl->valid or next; |
546 | $pl->valid or next; |
| 339 | |
547 | |
| 340 | eval { |
548 | eval { |
| 341 | if ($pl->{last_save} + $SAVE_TIMEOUT <= $cf::RUNTIME) { |
549 | if ($pl->{last_save} + $SAVE_TIMEOUT <= $cf::RUNTIME) { |
|
|
550 | cf::wait_for_tick_begin; |
| 342 | $pl->save; |
551 | $pl->save; |
| 343 | Coro::Timer::sleep $SAVE_INTERVAL; |
552 | |
|
|
553 | unless ($pl->active) { |
|
|
554 | # check refcounts, this is tricky and needs to be adjusted to fit server internals |
|
|
555 | my $ob = $pl->ob; |
|
|
556 | Scalar::Util::weaken $pl; |
|
|
557 | Scalar::Util::weaken $ob; |
|
|
558 | my $a_ = $pl->refcnt;#d# |
|
|
559 | my $b_ = $ob->refcnt;#d# |
|
|
560 | my $pl_ref = $pl->refcnt_cnt; |
|
|
561 | my $ob_ref = $ob->refcnt_cnt; |
|
|
562 | |
|
|
563 | ## pl_ref == one from object + one from cf::PLAYER |
|
|
564 | ## ob_ref == one from simply being an object |
|
|
565 | if ($pl_ref == 2 && $ob_ref == 1) { |
|
|
566 | warn "player-scheduler destroy ", $ob->name;#d# |
|
|
567 | |
|
|
568 | # remove from sight and get fresh "copies" |
|
|
569 | $pl = delete $cf::PLAYER{$ob->name}; |
|
|
570 | $ob = $pl->ob; |
|
|
571 | |
|
|
572 | $ob->destroy; |
|
|
573 | $pl->destroy; |
|
|
574 | } else { |
|
|
575 | warn "player-scheduler refcnt ", $ob->name, " $pl_ref,$a_ $ob_ref,$b_\n";#d# |
|
|
576 | } |
|
|
577 | } |
| 344 | } |
578 | } |
| 345 | }; |
579 | }; |
| 346 | warn $@ if $@; |
580 | warn $@ if $@; |
| 347 | Coro::cede; |
581 | Coro::cede; |
| 348 | }; |
582 | }; |
