location-importer.in: Fix dangling variable

Message ID 452c5644-c309-78d0-3d3d-2ef123b08805@ipfire.org
State Accepted
Commit 00beb796d3c684d3e828c74b8d0c9fcace9bef4f
Headers
Series location-importer.in: Fix dangling variable |

Commit Message

Peter Müller Aug. 14, 2022, 4:02 p.m. UTC
  This fixes:

Traceback (most recent call last):
  File "/usr/bin/location-importer", line 1607, in <module>
    main()
  File "/usr/bin/location-importer", line 1605, in main
    c.run()
  File "/usr/bin/location-importer", line 140, in run
    ret = args.func(args)
  File "/usr/bin/location-importer", line 1234, in handle_update_overrides
    self._update_overrides_for_spamhaus_drop()
  File "/usr/bin/location-importer", line 1504, in _update_overrides_for_spamhaus_drop
    for sline in t.readlines():
NameError: name 't' is not defined

Signed-off-by: Peter Müller <peter.mueller@ipfire.org>
---
 src/scripts/location-importer.in | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
  

Comments

Michael Tremer Aug. 14, 2022, 4:05 p.m. UTC | #1
Thx. Merged.

> On 14 Aug 2022, at 17:02, Peter Müller <peter.mueller@ipfire.org> wrote:
> 
> This fixes:
> 
> Traceback (most recent call last):
> File "/usr/bin/location-importer", line 1607, in <module>
>   main()
> File "/usr/bin/location-importer", line 1605, in main
>   c.run()
> File "/usr/bin/location-importer", line 140, in run
>   ret = args.func(args)
> File "/usr/bin/location-importer", line 1234, in handle_update_overrides
>   self._update_overrides_for_spamhaus_drop()
> File "/usr/bin/location-importer", line 1504, in _update_overrides_for_spamhaus_drop
>   for sline in t.readlines():
> NameError: name 't' is not defined
> 
> Signed-off-by: Peter Müller <peter.mueller@ipfire.org>
> ---
> src/scripts/location-importer.in | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/src/scripts/location-importer.in b/src/scripts/location-importer.in
> index 1785791..8d47497 100644
> --- a/src/scripts/location-importer.in
> +++ b/src/scripts/location-importer.in
> @@ -1499,7 +1499,7 @@ class CLI(object):
> 			# Iterate through every line, filter comments and add remaining ASNs to
> 			# the override table in case they are valid...
> 			with self.db.transaction():
> -				for sline in t.readlines():
> +				for sline in f.readlines():
> 					# The response is assumed to be encoded in UTF-8...
> 					sline = sline.decode("utf-8")
> 
> -- 
> 2.35.3
  

Patch

diff --git a/src/scripts/location-importer.in b/src/scripts/location-importer.in
index 1785791..8d47497 100644
--- a/src/scripts/location-importer.in
+++ b/src/scripts/location-importer.in
@@ -1499,7 +1499,7 @@  class CLI(object):
 			# Iterate through every line, filter comments and add remaining ASNs to
 			# the override table in case they are valid...
 			with self.db.transaction():
-				for sline in t.readlines():
+				for sline in f.readlines():
 					# The response is assumed to be encoded in UTF-8...
 					sline = sline.decode("utf-8")