Merge lp:~ari-tczew/ubuntu/karmic/agg/lp-393923 into lp:ubuntu/karmic/agg

Proposed by Artur Rona
Status: Merged
Merge reported by: Stefano Rivera
Merged at revision: not available
Proposed branch: lp:~ari-tczew/ubuntu/karmic/agg/lp-393923
Merge into: lp:ubuntu/karmic/agg
Diff against target: 76 lines (+35/-3)
4 files modified
debian/changelog (+9/-0)
debian/patches/00list (+1/-0)
debian/patches/06_fix_recursion_crash.dpatch (+3/-3)
debian/patches/07_fix_missing_includes.dpatch (+22/-0)
To merge this branch: bzr merge lp:~ari-tczew/ubuntu/karmic/agg/lp-393923
Reviewer Review Type Date Requested Status
Ubuntu Development Team Pending
Review via email: mp+33444@code.launchpad.net
To post a comment you must log in.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'debian/changelog'
--- debian/changelog 2009-04-02 18:39:54 +0000
+++ debian/changelog 2010-08-23 20:18:45 +0000
@@ -1,3 +1,12 @@
1agg (2.5+dfsg1-1ubuntu1.9.10.1) karmic-proposed; urgency=low
2
3 * debian/patches/07_fix_missing_includes.dpatch: Fix missing
4 includes causes compilation problem. (LP: #393923, Closes: #575620)
5 * debian/patches/06_fix_recursion_crash.dpatch: Update patch,
6 it's necessary to fix compilation problem.
7
8 -- Artur Rona <ari-tczew@tlen.pl> Mon, 23 Aug 2010 01:34:28 +0200
9
1agg (2.5+dfsg1-1ubuntu1) jaunty; urgency=low10agg (2.5+dfsg1-1ubuntu1) jaunty; urgency=low
211
3 [ Andrew Straw ]12 [ Andrew Straw ]
413
=== modified file 'debian/patches/00list'
--- debian/patches/00list 2009-04-02 18:39:54 +0000
+++ debian/patches/00list 2010-08-23 20:18:45 +0000
@@ -4,3 +4,4 @@
404_no_rpath404_no_rpath
505_x11_makefile_am.dpatch505_x11_makefile_am.dpatch
606_fix_recursion_crash.dpatch606_fix_recursion_crash.dpatch
707_fix_missing_includes.dpatch
78
=== modified file 'debian/patches/06_fix_recursion_crash.dpatch'
--- debian/patches/06_fix_recursion_crash.dpatch 2009-04-02 18:39:54 +0000
+++ debian/patches/06_fix_recursion_crash.dpatch 2010-08-23 20:18:45 +0000
@@ -3,7 +3,7 @@
3##3##
4## All lines beginning with `## DP:' are a description of the patch.4## All lines beginning with `## DP:' are a description of the patch.
5## DP: Ubuntu: http://launchpad.net/bugs/3448495## DP: Ubuntu: http://launchpad.net/bugs/344849
6## DP: Patch: https://trac.mapnik.org/ticket/2536## DP: Patch: https://trac.mapnik.org/ticket/253, http://trac.mapnik.org/changeset/1801
7## DP: Description: Avoid a crash caused by an infinite recursion when drawing extremely long lines7## DP: Description: Avoid a crash caused by an infinite recursion when drawing extremely long lines
88
9@DPATCH@9@DPATCH@
@@ -16,8 +16,8 @@
16 int cy = (y1 + y2) >> 1;16 int cy = (y1 + y2) >> 1;
17+17+
18+ // Bail if values are so large they are likely to wrap18+ // Bail if values are so large they are likely to wrap
19+ if ((abs(x1) >= INT_MAX/2) || (abs(y1) >= INT_MAX/2) ||19+ if ((std::abs(x1) >= std::numeric_limits<int>::max()/2) || (std::abs(y1) >= std::numeric_limits<int>::max()/2) ||
20+ (abs(x2) >= INT_MAX/2) || (abs(y2) >= INT_MAX/2))20+ (std::abs(x2) >= std::numeric_limits<int>::max()/2) || (std::abs(y2) >= std::numeric_limits<int>::max()/2))
21+ return;21+ return;
22+22+
23 line(x1, y1, cx, cy);23 line(x1, y1, cx, cy);
2424
=== added file 'debian/patches/07_fix_missing_includes.dpatch'
--- debian/patches/07_fix_missing_includes.dpatch 1970-01-01 00:00:00 +0000
+++ debian/patches/07_fix_missing_includes.dpatch 2010-08-23 20:18:45 +0000
@@ -0,0 +1,22 @@
1#! /bin/sh /usr/share/dpatch/dpatch-run
2## From: Artur Rona
3## Description: Add missing includes for fix compile.
4## Bug: https://launchpad.net/bugs/393923
5## Origin: upstream, https://trac.mapnik.org/changeset/1801
6## Author: Dane Springmeyer
7
8@DPATCH@
9
10diff -pruN -x '*~' agg-2.5+dfsg1.orig/include/agg_rasterizer_cells_aa.h agg-2.5+dfsg1/include/agg_rasterizer_cells_aa.h
11--- agg-2.5+dfsg1.orig/include/agg_rasterizer_cells_aa.h 2007-10-11 00:06:16.000000000 +0200
12+++ agg-2.5+dfsg1/include/agg_rasterizer_cells_aa.h 2010-08-23 01:51:07.000000000 +0200
13@@ -40,7 +40,8 @@
14 #define AGG_RASTERIZER_CELLS_AA_INCLUDED
15
16 #include <string.h>
17-#include <math.h>
18+#include <cstdlib>
19+#include <limits>
20 #include "agg_math.h"
21 #include "agg_array.h"
22

Subscribers

People subscribed via source and target branches

to all changes: