[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#685788: marked as done (unblock: vite/1.2+svn1347-2)



Your message dated Fri, 24 Aug 2012 16:55:17 +0200
with message-id <503795D5.6010007@thykier.net>
and subject line Re: Bug#685788: unblock: vite/1.2+svn1347-2
has caused the Debian Bug report #685788,
regarding unblock: vite/1.2+svn1347-2
to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact owner@bugs.debian.org
immediately.)


-- 
685788: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=685788
Debian Bug Tracking System
Contact owner@bugs.debian.org with problems
--- Begin Message ---
Package: release.debian.org
Severity: normal
User: release.debian.org@packages.debian.org
Usertags: unblock

Please unblock package vite

The difference between testing vite/1.2+svn1347-1 and sid
vite/1.2+svn1347-2 is an upstream patch fixing a crash on big line
number (>= 1 million), as can be seen below.

unblock vite/1.2+svn1347-2

--- a/src/common/Errors.cpp.original	2012-08-03 22:17:21.687362090 +0200
+++ b/src/common/Errors.cpp	2012-08-03 22:19:56.516207713 +0200
@@ -118,8 +118,8 @@
 }
 
 void Error::set(const string kind_of_error, const unsigned int line_number, const int priority){
-    char line[10];
-    sprintf(line, "%d : ", line_number);
+    char line[sizeof(line_number)*3+3+1];
+    snprintf(line, sizeof(line), "%d : ", line_number);
     set(line + kind_of_error, priority);
 }
 
@@ -129,8 +129,8 @@
 }
 
 void Error::set_and_print(const string kind_of_error, const unsigned int line_number, const int priority){
-    char line[10];
-    sprintf(line, "%d", line_number);
+    char line[sizeof(line_number)*3+1];
+    snprintf(line, sizeof(line), "%d", line_number);
     set(kind_of_error + " on line " + line, priority);
     print(priority);
 }
@@ -138,8 +138,8 @@
 
 bool Error::set_if(bool condition, const string kind_of_error, const unsigned int line_number, const int priority){
     if(condition){
-        char line[10];
-        sprintf(line, "%d", line_number);
+        char line[sizeof(line_number)*3+1];
+        snprintf(line, sizeof(line), "%d", line_number);
         set(kind_of_error + " on line " + line, priority);
         return true;
     }

-- System Information:
Debian Release: wheezy/sid
  APT prefers testing
  APT policy: (990, 'testing'), (500, 'unstable'), (500, 'stable'), (1, 'experimental')
Architecture: amd64 (x86_64)
Foreign Architectures: i386

Kernel: Linux 3.0.4 (SMP w/8 CPU cores)
Locale: LANG=fr_FR.UTF-8, LC_CTYPE=fr_FR.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/bash

-- 
Samuel Thibault <samuel.thibault@ens-lyon.org>
requests.</FONT></SPAN></TD></TR></TBODY></TABLE></DIV></BODY></HTML>agnjo
gj a po  mi
shnthdrdcvallus hsx mvgduwolgfwtq
uzuy
s
p
h
 -+- spams forever ... -+- 

--- End Message ---
--- Begin Message ---
On 2012-08-24 16:40, Samuel Thibault wrote:
> Package: release.debian.org
> Severity: normal
> User: release.debian.org@packages.debian.org
> Usertags: unblock
> 
> Please unblock package vite
> 
> The difference between testing vite/1.2+svn1347-1 and sid
> vite/1.2+svn1347-2 is an upstream patch fixing a crash on big line
> number (>= 1 million), as can be seen below.
> 
> unblock vite/1.2+svn1347-2
> 
> [...]

Unblocked, thanks.

~Niels

--- End Message ---

Reply to: