[Debian-l10n-commits] [ddtp] 03/04: Fix UTF-8 check now Encode::decode_utf8 apparently clobbers argument.

Martijn van Oosterhout kleptog-guest at moszumanska.debian.org
Sat Jan 18 16:06:00 UTC 2014


This is an automated email from the git hooks/post-receive script.

kleptog-guest pushed a commit to branch master
in repository ddtp.

commit 860e563a1307456a64e039b0d4c881ea9f754a87
Author: Martijn van Oosterhout <kleptog at svana.org>
Date:   Sat Jan 18 15:59:34 2014 +0000

    Fix UTF-8 check now Encode::decode_utf8 apparently clobbers argument.
---
 ddtss/ddtss-cgi | 14 ++++++++++++++
 1 file changed, 14 insertions(+)

diff --git a/ddtss/ddtss-cgi b/ddtss/ddtss-cgi
index 9212b37..c5e841e 100755
--- a/ddtss/ddtss-cgi
+++ b/ddtss/ddtss-cgi
@@ -1041,7 +1041,9 @@ sub forexternalreview_display
   { display_error( "Fetched description didn't contain package name" ) }
   
   eval {
+    my $temp = $data;
     my $str = Encode::decode_utf8($data, Encode::FB_CROAK);
+    $data = $temp;
   };
   if($@)
   {
@@ -1456,8 +1458,12 @@ sub submit_externalreview
   $long =~ s/^/ /mg; # Insert leading space
 
   eval {
+    my $tempshort = $short;
+    my $templong = $long;
     my $shortx = Encode::decode_utf8($short, Encode::FB_CROAK);
     my $longx = Encode::decode_utf8($long, Encode::FB_CROAK);
+    $short = $tempshort;
+    $long = $templong;
     
     if( length($shortx) > 80 or $longx =~ /^.{80}/m )
     {
@@ -1498,7 +1504,9 @@ sub submit_externalreview
   { return( "Fetched description didn't contain package name" ) }
   
   eval {
+    my $temp = $data;
     my $str = Encode::decode_utf8($data, Encode::FB_CROAK);
+    $data = $temp;
   };
   if($@)
   {
@@ -1568,8 +1576,12 @@ sub submit_translation
   }
   
   eval {
+    my $tempshort = $short;
+    my $templong = $long;
     my $shortx = Encode::decode_utf8($short, Encode::FB_CROAK);
     my $longx = Encode::decode_utf8($long, Encode::FB_CROAK);
+    $short = $tempshort;
+    $long = $templong;
     
     if( length($shortx) > 80 or $longx =~ /^.{80}/m )
     {
@@ -1752,7 +1764,9 @@ sub fetch_translation
   $package = $fetch_package;
   
   eval {
+    my $temp = $data;
     my $str = Encode::decode_utf8($data, Encode::FB_CROAK);
+    $data = $temp;
   };
   if($@)
   {

-- 
Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/debian-l10n/ddtp.git



More information about the Debian-l10n-commits mailing list