bounce invalid pc41s back

[spider.git] / perl / DXProtHandle.pm

diff --git a/perl/DXProtHandle.pm b/perl/DXProtHandle.pm
index 2dd97bc051690c793deddf57bcc22355ef662431..641299d36c5b126f2abf40f7769b9c27b81c9e3d 100644 (file)
--- a/perl/DXProtHandle.pm
+++ b/perl/DXProtHandle.pm
@@ -540,27 +540,23 @@ sub handle_17
 
        RouteDB::delete($ncall, $self->{call});
 
-       unless ($ncall eq $self->{call}) {
-               dbg("PCPROT: PC17 from non-local $ncall, ignored") if isdbg('chanerr');
-               return;
-       }
-
        my $uref = Route::User::get($ucall);
        unless ($uref) {
                dbg("PCPROT: Route::User $ucall not in config") if isdbg('chanerr');
+               return;
        }
        my $parent = Route::Node::get($ncall);
        unless ($parent) {
                dbg("PCPROT: Route::Node $ncall not in config") if isdbg('chanerr');
+               return;
        }                       
 
-       $dxchan = $parent->dxchan if $parent;
+       $dxchan = DXChannel::get($ncall);
        if ($dxchan && $dxchan ne $self) {
                dbg("PCPROT: PC17 from $self->{call} trying to alter locally connected $ncall, ignored!") if isdbg('chanerr');
                return;
        }
 
-       $dxchan = DXChannel::get($ncall);
        unless ($dxchan) {
                if ($parent->via_pc92) {
                        dbg("PCPROT: non-local node controlled by PC92, ignored") if isdbg('chanerr');
@@ -574,12 +570,8 @@ sub handle_17
        }
 
        # input filter if required and then remove user if present
-       if ($parent) {
 #              return unless $self->in_filter_route($parent);  
-               $parent->del_user($uref) if $uref;
-       } else {
-               $parent = Route->new($ncall);  # throw away
-       }
+       $parent->del_user($uref);
 
        # send info to all logged in thingies
        $self->tell_login('logoutu', "$ncall: $ucall") if DXUser->get_current($ncall)->is_local_node;
@@ -590,7 +582,6 @@ sub handle_17
                return;
        }
 
-       $uref = Route->new($ucall) unless $uref; # throw away
        $self->route_pc17($origin, $line, $parent, $uref);
 #      $self->route_pc92d($main::mycall, undef, $parent, $uref) if $dxchan;
 }
@@ -617,7 +608,8 @@ sub handle_18
                }
                $self->{handle_xml}++ if DXXml::available() && $_[1] =~ /\bxml/;
                if ($_[1] =~ /\bpc9x/) {
-                       $self->{do_pc9x}++;
+                       $self->{do_pc9x} = 1;
+                       dbg("Do px9x set on $self->{call}");
                }
        } else {
                $self->version(50.0);
@@ -1154,6 +1146,13 @@ sub handle_41
        $user = DXUser->new($call) unless $user;
                        
        if ($_[2] == 1) {
+               if (($_[3] =~ /spotter/i || $_[3] =~ /self/i) && $user->name && $user->name ne $_[3]) {
+                       dbg("PCPROT: invalid name") if isdbg('chanerr');
+                       if ($main::mycall eq 'GB7DJK' || $main::mycall eq 'GB7BAA' || $main::mycall eq 'WR3D') {
+                               DXChannel::broadcast_nodes(pc41($_[1], 1, $user->name)); # send it to everyone including me
+                       }
+                       return;
+               }
                $user->name($_[3]);
        } elsif ($_[2] == 2) {
                $user->qth($_[3]);
@@ -1468,8 +1467,6 @@ sub handle_92
 
        my (@radd, @rdel);
        
-       $self->{do_pc9x} ||= 1;
-
        my $pcall = $_[1];
        unless ($pcall) {
                dbg("PCPROT: invalid callsign string '$_[1]', ignored") if isdbg('chanerr');
@@ -1486,12 +1483,17 @@ sub handle_92
 
        if ($pcall eq $self->{call} && $self->{state} eq 'init') {
                $self->state('init92');
+               $self->{do_pc9x} = 1;
+               dbg("Do pc9x set on $pcall");
+       } 
+       unless ($self->{do_pc9x}) {
+               dbg("PCPROT: PC9x come in from non-PC9x node, ignored") if isdbg('chanerr');
+               return;
        }
 
        my $parent = check_pc9x_t($pcall, $t, 92, 1) || return;
        my $oparent = $parent;
        
-       $parent->lastid->{92} = $t;
        $parent->do_pc9x(1);
        $parent->via_pc92(1);
 
@@ -1616,7 +1618,7 @@ sub handle_93
        my $line = shift;
        my $origin = shift;
 
-       $self->{do_pc9x} ||= 1;
+#      $self->{do_pc9x} ||= 1;
 
        my $pcall = $_[1];
        unless (is_callsign($pcall)) {