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 | |
7 | |
8 | my $PLAYERDIR = sprintf "%s/%s", cf::localdir, cf::playerdir; |
8 | our $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 |
11 | sub nuke_str { |
11 | sub 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 | |
15 | sub query { |
15 | sub query { |
16 | my ($ns, $flags, $text) = @_; |
16 | my ($ns, $flags, $text) = @_; |
17 | |
17 | |
18 | my $current = $Coro::current; |
18 | my $current = $Coro::current; |
19 | $ns->query ($flags, $text, sub { $current->ready; $current = $_[0]; }); |
19 | $ns->query ($flags, $text, sub { $current->ready; $current = $_[0] }); |
20 | Coro::schedule while ref $current; |
20 | Coro::schedule while ref $current; |
21 | |
21 | |
22 | $current |
22 | $current |
23 | } |
23 | } |
24 | |
24 | |
… | |
… | |
40 | return unless cf::player::find_active $user; |
40 | return unless cf::player::find_active $user; |
41 | |
41 | |
42 | $ns->send_drawinfo ( |
42 | $ns->send_drawinfo ( |
43 | "That player is already logged in on this server. " |
43 | "That player is already logged in on this server. " |
44 | . "If you want to create a new player, choose another name. " |
44 | . "If you want to create a new player, choose another name. " |
45 | . "If you are already a registered player, make sure nobody " |
45 | . "If you have already a registered, make sure nobody " |
46 | . "else is using your account at this time. If you lost your conenction " |
46 | . "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 " |
47 | . "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 " |
48 | . "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, " |
49 | . "you do not have another client running. If you use windows, reboot, " |
50 | . "this will fix anything.", |
50 | . "this will fix anything.", |
51 | cf::NDI_RED |
51 | cf::NDI_RED |
52 | ); |
52 | ); |
53 | |
53 | |
54 | 1 |
54 | 1 |
55 | } |
55 | } |
56 | |
56 | |
57 | sub check_clean_save { |
57 | sub safe_spot($) { |
58 | my ($pl) = @_; |
58 | my ($pl) = @_; |
|
|
59 | |
|
|
60 | my $ob = $pl->ob; |
59 | |
61 | |
60 | unless (delete $pl->{clean_save}) { |
62 | my $m = $ob->map |
61 | #d#TODO |
63 | or return; |
|
|
64 | my $x = $ob->x; |
|
|
65 | my $y = $ob->y; |
|
|
66 | |
|
|
67 | # return 0;#d# |
|
|
68 | # warn join ":", $m->at ($x, $y);#d# |
|
|
69 | # warn "FOO$m { ".scalar ($m->at ($x, $y))." }\n"; |
|
|
70 | # return 0; |
|
|
71 | |
|
|
72 | scalar grep $_->type == cf::SAVEBED, $m->at ($x, $y) |
|
|
73 | } |
|
|
74 | |
|
|
75 | sub enter_map { |
|
|
76 | my ($pl) = @_; |
|
|
77 | |
|
|
78 | my $ob = $pl->ob; |
|
|
79 | |
|
|
80 | my ($map, $x, $y) |
|
|
81 | = $ob->{_link_pos} |
|
|
82 | ? @{delete $ob->{_link_pos}} |
|
|
83 | : ($pl->maplevel, $ob->x, $ob->y); |
|
|
84 | |
|
|
85 | $ob->enter_link; |
|
|
86 | |
|
|
87 | my $m = cf::map::find $map; |
|
|
88 | my $time = delete $pl->{unclean_save}; |
|
|
89 | |
|
|
90 | if ($time && $m) { |
|
|
91 | if ($time < $m->{instantiate_time}) { |
|
|
92 | # the map was reset in the meantime |
|
|
93 | my $age = $cf::RUNTIME - $time; |
|
|
94 | |
|
|
95 | warn $ob->name, " map reset after logout, logout age $age (>= $MAX_DISCONNECT_TIME)\n";#d# |
|
|
96 | |
|
|
97 | ($map, $x, $y) = $pl->savebed; |
|
|
98 | |
|
|
99 | if ($age >= $MAX_DISCONNECT_TIME) { |
|
|
100 | $ob->message ( |
|
|
101 | "You didn't use a bed to reality to leave this realm, leaving your body in great danger. " |
|
|
102 | . "Unfortunately, nobody was near to help you when the monsters arrived to eat you. " |
|
|
103 | . "Maybe you can find comfort in the thought that your body was quite satisfying in taste... " |
|
|
104 | . "H<You disconnected too long without having used a savebed.>", |
|
|
105 | cf::NDI_RED |
|
|
106 | ); |
|
|
107 | # kill them. |
|
|
108 | # reminds me of the famous badness 10000 syndrome... |
|
|
109 | $ob->stats->hp (-10000); #] if they survive this they deserved to live |
|
|
110 | $pl->killer ("a cave-in"); |
|
|
111 | } else { |
|
|
112 | $ob->message ( |
|
|
113 | "You didn't use a bed to reality to leave this realm, leaving your body in great danger. " |
|
|
114 | . "Fortunately, some friendly dwellers found you, checked your passport, and brought you to safety. " |
|
|
115 | . "Better use a savebed next time, much worse things could have happened... " |
|
|
116 | . "H<You disconnected without having used a savebed. When you do that for too long, you might die.>", |
|
|
117 | cf::NDI_RED |
|
|
118 | ); |
|
|
119 | } |
|
|
120 | } else { |
|
|
121 | $ob->message ( |
|
|
122 | "You didn't use a bed to reality to leave this realm. This is very dangerous, " |
|
|
123 | . "as lots of things could happen when you leave by other means, such as cave-ins, " |
|
|
124 | . "or monsters suddenly snapping your body. Better use a savebed next time. " |
|
|
125 | . "H<Always apply a bed of reality to disconnect from the server.>", |
|
|
126 | cf::NDI_RED |
|
|
127 | ); |
|
|
128 | } |
62 | } |
129 | } |
|
|
130 | |
|
|
131 | $ob->goto ($map, $x, $y); |
63 | } |
132 | } |
64 | |
133 | |
65 | # delete a player directory, be non-blocking AND synchronous... |
134 | # delete a player directory, be non-blocking AND synchronous... |
66 | # (thats hard, so we crap out and fork). |
135 | # (thats hard, so we crap out and fork). |
67 | sub nuke_playerdir { |
136 | sub nuke_playerdir { |
68 | my ($user) = @_; |
137 | my ($user) = @_; |
69 | |
138 | |
70 | aio_stat "$PLAYERDIR/$user"; |
139 | my $temp = "$PLAYERDIR/~$Coro::current~deleting~"; |
71 | system "cd \Q$PLAYERDIR\E " |
|
|
72 | . "&& mv \Q$user\E ~\Q$Coro::current\E~deleting~ 2>/dev/null " |
|
|
73 | . "&& (rm -rf ~\Q$Coro::current\E~deleting~ &)"; |
|
|
74 | } |
|
|
75 | |
140 | |
76 | sub addme { |
141 | cf::fork_call { |
|
|
142 | rename "$PLAYERDIR/$user", $temp; |
|
|
143 | system "rm", "-rf", $temp; |
|
|
144 | }; |
|
|
145 | } |
|
|
146 | |
|
|
147 | cf::client->attach (on_addme => sub { |
77 | my ($ns) = @_; |
148 | my ($ns) = @_; |
78 | |
149 | |
79 | $ns->destroy if $ns->pl; |
150 | if (!$ns->facecache) |
|
|
151 | { |
|
|
152 | $ns->send_drawinfo (<<EOF, cf::NDI_RED); |
|
|
153 | |
|
|
154 | |
|
|
155 | *** |
|
|
156 | *** WARNING: |
|
|
157 | *** Your client does not support face/image caching, |
|
|
158 | *** or it has been disabled. Face caching is mandatory |
|
|
159 | *** so please enable it or use a newer client. |
|
|
160 | *** |
|
|
161 | *** Look at your client preferences: |
|
|
162 | *** |
|
|
163 | *** deliantra: all known versions automatically enable the facecache. |
|
|
164 | *** cfclient: use the -cache commandline option. |
|
|
165 | *** cfclient: map will not redraw automatically (bug). |
|
|
166 | *** gcfclient: use -cache commandline option, or enable |
|
|
167 | *** gcfclient: Client => Configure => Map & Image => Cache Images. |
|
|
168 | *** jcrossclient: your client is broken, use deliantra or gcfclient. |
|
|
169 | *** |
|
|
170 | *** |
|
|
171 | EOF |
|
|
172 | if ($ns->version =~ /jcrossclient/) { |
|
|
173 | # let them, for now |
|
|
174 | } else { |
|
|
175 | $ns->flush; |
|
|
176 | return $ns->destroy; |
|
|
177 | } |
|
|
178 | |
|
|
179 | # $ns->facecache = true; |
|
|
180 | } |
|
|
181 | |
|
|
182 | if ($ns->mapmode < cf::Map1aCmd) { |
|
|
183 | $ns->send_drawinfo (<<EOF, cf::NDI_RED); |
|
|
184 | |
|
|
185 | |
|
|
186 | *** |
|
|
187 | *** WARNING: |
|
|
188 | *** Your client is too old. Please upgrade to a newer version. |
|
|
189 | EOF |
|
|
190 | |
|
|
191 | $ns->flush; |
|
|
192 | return $ns->destroy; |
|
|
193 | } |
|
|
194 | |
|
|
195 | $ns->pl and return $ns->destroy; |
80 | |
196 | |
81 | $ns->async (sub { |
197 | $ns->async (sub { |
|
|
198 | $Coro::current->{desc} = "addme init"; |
|
|
199 | |
82 | my ($user, $pass); |
200 | my ($user, $pass); |
83 | |
201 | |
84 | $ns->send_packet ("addme_success"); |
202 | $ns->send_packet ("addme_success"); |
85 | |
203 | |
86 | for (;;) { |
204 | for (;;) { |
… | |
… | |
99 | $ns->send_drawinfo ( |
217 | $ns->send_drawinfo ( |
100 | "That username is currently used in another login session. " |
218 | "That username is currently used in another login session. " |
101 | . "Chose another, or wait till the other session has ended.", |
219 | . "Chose another, or wait till the other session has ended.", |
102 | cf::NDI_RED |
220 | cf::NDI_RED |
103 | ); |
221 | ); |
104 | } elsif ($user =~ /^[a-zA-Z0-9][a-zA-Z0-9\-_]{2,17}$/) { |
222 | } elsif ($user =~ /^[a-zA-Z0-9][a-zA-Z0-9\-_]{2,17}\z/) { |
105 | last; |
223 | last; |
106 | } else { |
224 | } else { |
107 | $ns->send_drawinfo ( |
225 | $ns->send_drawinfo ( |
108 | "Your username contains illegal characters " |
226 | "Your username contains illegal characters " |
109 | . "(only a-z, A-Z and 0-9 are allowed), " |
227 | . "(only a-z, A-Z and 0-9 are allowed), " |
110 | . "or is not between 3 and 18 characters in length.", |
228 | . "or is not between 3 and 18 characters in length.", |
111 | cf::NDI_RED |
229 | cf::NDI_RED |
112 | ); |
230 | ); |
113 | } |
231 | } |
|
|
232 | Coro::Timer::sleep 0.4; |
114 | } |
233 | } |
115 | |
234 | |
116 | check_playing $ns, $user and next; |
235 | check_playing $ns, $user and next; |
|
|
236 | |
|
|
237 | $Coro::current->{desc} = "addme($user) pass"; |
117 | |
238 | |
118 | $ns->send_drawinfo ( |
239 | $ns->send_drawinfo ( |
119 | "Welcome $user, please enter your password now. " |
240 | "Welcome $user, please enter your password now. " |
120 | . "New users should now choose a password. " |
241 | . "New users should now choose a password. " |
121 | . "Anything your client lets you enter is fine.", |
242 | . "Anything your client lets you enter is fine.", |
… | |
… | |
129 | $ns->send_drawinfo ( |
250 | $ns->send_drawinfo ( |
130 | "Try to use at least three characters as your password please, " |
251 | "Try to use at least three characters as your password please, " |
131 | . "that cannot be too much to ask for :)", |
252 | . "that cannot be too much to ask for :)", |
132 | cf::NDI_RED |
253 | cf::NDI_RED |
133 | ); |
254 | ); |
|
|
255 | Coro::Timer::sleep 0.4; |
134 | } |
256 | } |
135 | |
257 | |
136 | # lock this username for the remainder of this login session |
258 | # lock this username for the remainder of this login session |
137 | if ($cf::LOGIN_LOCK{$user}) { |
259 | if ($cf::LOGIN_LOCK{$user}) { |
138 | $ns->send_drawinfo ( |
260 | $ns->send_drawinfo ( |
… | |
… | |
144 | } |
266 | } |
145 | local $cf::LOGIN_LOCK{$user} = 1; |
267 | local $cf::LOGIN_LOCK{$user} = 1; |
146 | |
268 | |
147 | check_playing $ns, $user and next; |
269 | check_playing $ns, $user and next; |
148 | |
270 | |
|
|
271 | $Coro::current->{desc} = "addme($user) check"; |
|
|
272 | |
149 | # try to read the user file and check the password |
273 | # try to read the user file and check the password |
150 | if (my $pl = cf::player::find $user) { |
274 | if (my $pl = cf::player::find $user) { |
151 | aio_stat $pl->path and next; |
275 | aio_stat $pl->path and next; |
152 | my $mtime = (stat _)[9]; |
276 | my $mtime = (stat _)[9]; |
153 | my $hash = $pl->password; |
277 | my $hash = $pl->password; |
… | |
… | |
155 | if ($cf::CFG{ext_login_nocheck} or $hash eq crypt $pass, $hash) { |
279 | if ($cf::CFG{ext_login_nocheck} or $hash eq crypt $pass, $hash) { |
156 | nuke_str $pass; |
280 | nuke_str $pass; |
157 | # password matches, wonderful |
281 | # password matches, wonderful |
158 | my $pl = cf::player::find $user or next; |
282 | my $pl = cf::player::find $user or next; |
159 | $pl->connect ($ns); |
283 | $pl->connect ($ns); |
160 | check_clean_save $pl; |
284 | enter_map $pl; |
161 | $pl->{clean_save} = 1; |
|
|
162 | last; |
285 | last; |
163 | } elsif (can_cleanup $pl, $mtime) { |
286 | } elsif (can_cleanup $pl, $mtime) { |
164 | Coro::Timer::sleep 1; |
287 | Coro::Timer::sleep 1; |
165 | |
288 | |
166 | $ns->send_drawinfo ( |
289 | $ns->send_drawinfo ( |
… | |
… | |
191 | . "(check for Numlock and other semi-obvious error sources).", |
314 | . "(check for Numlock and other semi-obvious error sources).", |
192 | cf::NDI_RED |
315 | cf::NDI_RED |
193 | ); |
316 | ); |
194 | next; |
317 | next; |
195 | } |
318 | } |
|
|
319 | } else { |
|
|
320 | # unable to load the playerfile: |
|
|
321 | # check wether the player dir exists, which means the file is corrupted or |
|
|
322 | # something very similar. |
|
|
323 | if (!aio_stat cf::player::playerdir $user) { |
|
|
324 | $ns->send_drawinfo ( |
|
|
325 | "Unable to retrieve this player. It might be a locked or broken account. " |
|
|
326 | . "If this is your account, ask a dungeon master for assistance. " |
|
|
327 | . "Otherwise choose a different login name.", |
|
|
328 | cf::NDI_RED |
|
|
329 | ); |
|
|
330 | next; |
|
|
331 | } |
196 | } |
332 | } |
197 | |
333 | |
198 | # the rest of this function is character creation |
334 | # the rest of this function is character creation |
|
|
335 | $Coro::current->{desc} = "addme($user) chargen"; |
199 | |
336 | |
200 | # just to make sure nothing is left over |
337 | # just to make sure nothing is left over |
201 | nuke_playerdir $user; |
338 | nuke_playerdir $user; |
202 | |
339 | |
203 | my $pass2 = query $ns, cf::CS_QUERY_HIDEINPUT, "Please type your password again."; |
340 | my $pass2 = query $ns, cf::CS_QUERY_HIDEINPUT, "Please type your password again."; |
… | |
… | |
207 | nuke_str $pass2; |
344 | nuke_str $pass2; |
208 | $ns->send_drawinfo ( |
345 | $ns->send_drawinfo ( |
209 | "The passwords do not match, please try again.", |
346 | "The passwords do not match, please try again.", |
210 | cf::NDI_RED |
347 | cf::NDI_RED |
211 | ); |
348 | ); |
|
|
349 | Coro::Timer::sleep 0.5; |
212 | next; |
350 | next; |
213 | } |
351 | } |
214 | |
352 | |
215 | nuke_str $pass2; |
353 | nuke_str $pass2; |
216 | |
354 | |
217 | my $pl = cf::player::new $user; |
355 | my $pl = cf::player::new $user; |
218 | $pl->password (crypt $pass, join '', ('.', '/', 0..9, 'A'..'Z', 'a'..'z')[rand 64, rand 64]); |
356 | $pl->password (crypt $pass, join '', ('.', '/', 0..9, 'A'..'Z', 'a'..'z')[rand 64, rand 64]); |
219 | nuke_str $pass; |
357 | nuke_str $pass; |
220 | $pl->connect ($ns); |
358 | $pl->connect ($ns); |
221 | |
|
|
222 | my $ob = $pl->ob; |
359 | my $ob = $pl->ob; |
|
|
360 | |
|
|
361 | $ob->goto ($pl->maplevel, $ob->x, $ob->y); |
223 | |
362 | |
224 | while () { |
363 | while () { |
225 | $ob->update_stats; |
364 | $ob->update_stats; |
226 | $pl->save_stats; |
365 | $pl->save_stats; |
227 | |
366 | |
… | |
… | |
237 | $ob->swap_stats ($res - 1, $swap - 1); |
376 | $ob->swap_stats ($res - 1, $swap - 1); |
238 | } |
377 | } |
239 | } else { |
378 | } else { |
240 | $ob->roll_stats; |
379 | $ob->roll_stats; |
241 | } |
380 | } |
|
|
381 | |
|
|
382 | Coro::Timer::sleep 0.2; |
242 | } |
383 | } |
243 | |
384 | |
244 | $ob->set_animation (2); |
385 | $ob->set_animation (2); |
245 | $ob->add_statbonus; |
386 | $ob->add_statbonus; |
246 | |
387 | |
247 | $ns->send_drawinfo ($ob->msg, cf::NDI_BLUE); |
388 | while () { |
248 | $ns->send_packet (sprintf "query %d %s", cf::CS_QUERY_SINGLECHAR, |
389 | $ns->send_msg ("chargen-race-title", ucfirst $pl->title, -1); |
|
|
390 | my $msg = $ob->msg; |
|
|
391 | $msg =~ s/(?<=\S)\n(?=\S)/ /g; |
|
|
392 | $ns->send_msg ("chargen-race-description", $msg, cf::NDI_BLUE); |
|
|
393 | |
|
|
394 | my $res = query $ns, cf::CS_QUERY_SINGLECHAR, |
249 | "Now choose a character.\nPress any key to change outlook.\nPress `d' when you're pleased.\n"); |
395 | "Now choose a character.\nPress any key to change outlook.\nPress `d' when you're pleased.\n"; |
250 | |
396 | |
251 | $ns->state (cf::ST_CHANGE_CLASS); |
397 | last if $res =~ /[dD]/; |
|
|
398 | |
|
|
399 | $pl->chargen_race_next; |
|
|
400 | Coro::Timer::sleep 0.2; |
|
|
401 | } |
|
|
402 | |
|
|
403 | $pl->chargen_race_done; |
|
|
404 | |
|
|
405 | while () { |
|
|
406 | my $res = query $ns, cf::CS_QUERY_SINGLECHAR, |
|
|
407 | "Now choose a gender.\nPress 'f' to become female, and 'm' to become male.\n"; |
|
|
408 | |
|
|
409 | if ($res =~ /^[fF]/) { |
|
|
410 | $pl->gender (1); |
|
|
411 | last; |
|
|
412 | } elsif ($res =~ /^[mM]/) { |
|
|
413 | $pl->gender (0); |
|
|
414 | last; |
|
|
415 | } |
|
|
416 | Coro::Timer::sleep 0.2; |
|
|
417 | } |
|
|
418 | |
|
|
419 | $ob->reply (undef, "Welcome to Deliantra!"); |
|
|
420 | |
252 | delete $pl->{deny_save};#d# too early |
421 | delete $pl->{deny_save}; |
253 | |
422 | |
254 | last; |
423 | last; |
255 | } |
424 | } |
256 | }); |
425 | }); |
257 | } |
426 | }); |
258 | |
427 | |
259 | cf::register_command quit => sub { |
428 | cf::register_command quit => sub { |
260 | my ($ob, $arg) = @_; |
429 | my ($ob, $arg) = @_; |
261 | |
430 | |
262 | $ob->reply (undef, |
431 | $ob->reply (undef, |
… | |
… | |
278 | } else { |
447 | } else { |
279 | $ob->reply (undef, |
448 | $ob->reply (undef, |
280 | "Ok, quitting, hope to see you again.", |
449 | "Ok, quitting, hope to see you again.", |
281 | cf::NDI_UNIQUE | cf::NDI_RED); |
450 | cf::NDI_UNIQUE | cf::NDI_RED); |
282 | $pl->ns->flush; |
451 | $pl->ns->flush; |
283 | $pl->quit_character; |
452 | cf::async { $pl->quit_character }; |
284 | } |
453 | } |
285 | }); |
454 | }); |
286 | }; |
455 | }; |
287 | |
456 | |
288 | cf::object->attach ( |
457 | cf::object->attach ( |
… | |
… | |
328 | |
497 | |
329 | if ($cleanly) { |
498 | if ($cleanly) { |
330 | $_->ob->message ("$name left the game.", cf::NDI_DK_ORANGE | cf::NDI_UNIQUE) for cf::player::list; |
499 | $_->ob->message ("$name left the game.", cf::NDI_DK_ORANGE | cf::NDI_UNIQUE) for cf::player::list; |
331 | } else { |
500 | } else { |
332 | $_->ob->message ("$name uncerimoniously disconnected.", cf::NDI_DK_ORANGE | cf::NDI_UNIQUE) for cf::player::list; |
501 | $_->ob->message ("$name uncerimoniously disconnected.", cf::NDI_DK_ORANGE | cf::NDI_UNIQUE) for cf::player::list; |
333 | delete $pl->{clean_save}; |
502 | $pl->{unclean_save} = $cf::RUNTIME |
|
|
503 | unless safe_spot $pl; |
334 | } |
504 | } |
335 | }, |
505 | }, |
336 | ); |
506 | ); |
337 | |
507 | |
338 | cf::client->attach ( |
|
|
339 | on_addme => \&addme, |
|
|
340 | ); |
|
|
341 | |
|
|
342 | ############################################################################# |
508 | ############################################################################# |
343 | |
509 | |
344 | our $SCHEDULE_INTERVAL = 10; # time the player scheduler sleeps between runs |
510 | our $SCHEDULE_INTERVAL = $cf::CFG{player_schedule_interval} || 10; # time the player scheduler sleeps between runs |
345 | our $SAVE_TIMEOUT = 20; # save players every n seconds |
511 | our $SAVE_TIMEOUT = $cf::CFG{player_save_interval} || 20; # save players every n seconds |
346 | our $SAVE_INTERVAL = 0.5; # save at max. one player every $SAVE_INTERVAL |
|
|
347 | |
512 | |
348 | our $SCHEDULER = cf::async_ext { |
513 | our $SCHEDULER = cf::async_ext { |
|
|
514 | $Coro::current->{desc} = "player scheduler"; |
|
|
515 | |
349 | while () { |
516 | while () { |
350 | Coro::Timer::sleep $SCHEDULE_INTERVAL; |
517 | Coro::EV::timer_once $SCHEDULE_INTERVAL; |
351 | |
518 | |
352 | # this weird form of iteration over values is used because |
519 | # this weird form of iteration over values is used because |
353 | # the hash changes underneath us frequently, and for |
520 | # the hash changes underneath us frequently, and for |
354 | # keeps a direct reference to the value without (in 5.8 perls) |
521 | # keeps a direct reference to the value without (in 5.8 perls) |
355 | # keeping a reference, so this is prone to crashes or worse. |
522 | # keeping a reference, so this is prone to crashes or worse. |
… | |
… | |
361 | |
528 | |
362 | eval { |
529 | eval { |
363 | if ($pl->{last_save} + $SAVE_TIMEOUT <= $cf::RUNTIME) { |
530 | if ($pl->{last_save} + $SAVE_TIMEOUT <= $cf::RUNTIME) { |
364 | $pl->save; |
531 | $pl->save; |
365 | |
532 | |
366 | unless ($pl->active) { |
533 | unless ($pl->active || $pl->ns) { |
367 | # check refcounts, this is tricky and needs to be adjusted to fit server internals |
534 | # check refcounts, this is tricky and needs to be adjusted to fit server internals |
368 | my $ob = $pl->ob; |
535 | my $ob = $pl->ob; |
369 | Scalar::Util::weaken $pl; |
536 | |
370 | Scalar::Util::weaken $ob; |
|
|
371 | my $a_ = $pl->refcnt;#d# |
|
|
372 | my $b_ = $ob->refcnt;#d# |
|
|
373 | my $pl_ref = $pl->refcnt_cnt; |
537 | my $pl_ref = $pl->refcnt_cnt; |
374 | my $ob_ref = $ob->refcnt_cnt; |
538 | my $ob_ref = $ob->refcnt_cnt; |
375 | |
539 | |
376 | ## pl_ref == one from object + one from cf::PLAYER |
540 | ## pl_ref == $pl + ob->contr + %cf::PLAYER |
377 | ## ob_ref == one from simply being an object |
541 | ## ob_ref == $ob + pl->observe + simply being an object |
378 | if ($pl_ref == 2 && $ob_ref == 1) { |
542 | if ($pl_ref == 3 && $ob_ref == 3) { |
379 | warn "player-scheduler destroy ", $ob->name;#d# |
543 | warn "player-scheduler destroy ", $ob->name;#d# |
380 | |
544 | |
381 | # remove from sight and get fresh "copies" |
545 | # remove from sight and get fresh "copies" |
382 | $pl = delete $cf::PLAYER{$ob->name}; |
546 | $pl = delete $cf::PLAYER{$ob->name}; |
383 | $ob = $pl->ob; |
547 | $ob = $pl->ob; |
384 | |
548 | |
385 | $ob->destroy; |
|
|
386 | $pl->destroy; |
549 | $pl->destroy; # destroys $ob |
387 | } else { |
550 | } else { |
|
|
551 | my $a_ = $pl->refcnt;#d# |
|
|
552 | my $b_ = $ob->refcnt;#d# |
|
|
553 | |
388 | warn "player-scheduler refcnt ", $ob->name, " $pl_ref,$a_ $ob_ref,$b_\n";#d# |
554 | warn "player-scheduler refcnt ", $ob->name, " pl $pl_ref ob $ob_ref (C pl $a_ ob $b_)\n";#d# |
389 | } |
555 | } |
390 | } |
556 | } |
391 | Coro::Timer::sleep $SAVE_INTERVAL; |
|
|
392 | } |
557 | } |
393 | }; |
558 | }; |
394 | warn $@ if $@; |
559 | warn $@ if $@; |
395 | Coro::cede; |
560 | Coro::cede; |
396 | }; |
561 | }; |