at least show an error for bad pkgparts instead of a hang, RT#9578
authorivan <ivan>
Wed, 11 Aug 2010 21:53:56 +0000 (21:53 +0000)
committerivan <ivan>
Wed, 11 Aug 2010 21:53:56 +0000 (21:53 +0000)
FS/FS/cust_main/Import.pm

index 901ff18..e2165a0 100644 (file)
@@ -328,6 +328,12 @@ sub batch_import {
     tie my %hash, 'Tie::RefHash'; #this part is important
 
     if ( $cust_pkg{'pkgpart'} ) {
+
+      unless ( $cust_pkg{'pkgpart'} =~ /^\d+$/ ) {
+        $dbh->rollback if $oldAutoCommit;
+        return 'illegal pkgpart: '. $cust_pkg{'pkgpart'};
+      }
+
       my $cust_pkg = new FS::cust_pkg ( \%cust_pkg );
 
       my @svc_x = ();