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

Bug#694662: marked as done (unblock: libinfinity/0.5.2-6)



Your message dated Wed, 28 Nov 2012 20:08:25 +0000
with message-id <1354133305.27320.11.camel@jacala.jungle.funky-badger.org>
and subject line Re: Bug#694662: unblock: libinfinity/0.5.2-6
has caused the Debian Bug report #694662,
regarding unblock: libinfinity/0.5.2-6
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.)


-- 
694662: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=694662
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 libinfinity

0.5.2-6 fixes a crash of gobby-0.5 when the connection to the gobby server is
slow and if some text is selected towards the end of the buffer. This caused
crashes at both DebConf11 and at a recent BSP.

The change is pretty minimal.

unblock libinfinity/0.5.2-6
diff -Nru libinfinity-0.5.2/debian/changelog libinfinity-0.5.2/debian/changelog
--- libinfinity-0.5.2/debian/changelog	2012-10-24 00:08:40.000000000 +0200
+++ libinfinity-0.5.2/debian/changelog	2012-11-27 14:32:21.000000000 +0100
@@ -1,3 +1,11 @@
+libinfinity (0.5.2-6) unstable; urgency=low
+
+  * Add a patch that fixes a crash when trying to show a selection
+    towards the end of the document while this part is not available
+    yet. (Closes: #635712)
+
+ -- Philipp Kern <pkern@debian.org>  Tue, 27 Nov 2012 14:28:02 +0100
+
 libinfinity (0.5.2-5) unstable; urgency=low
 
   * Fix a crash when a connection goes down while attempting to
diff -Nru libinfinity-0.5.2/debian/patches/0001-Fix-assertion-fail-at-drawing-selection.patch libinfinity-0.5.2/debian/patches/0001-Fix-assertion-fail-at-drawing-selection.patch
--- libinfinity-0.5.2/debian/patches/0001-Fix-assertion-fail-at-drawing-selection.patch	1970-01-01 01:00:00.000000000 +0100
+++ libinfinity-0.5.2/debian/patches/0001-Fix-assertion-fail-at-drawing-selection.patch	2012-11-27 14:32:47.000000000 +0100
@@ -0,0 +1,48 @@
+From b4321f8ae838809fc9bfee324e554415d5869f4e Mon Sep 17 00:00:00 2001
+From: Armin Burgmeier <armin@arbur.net>
+Date: Mon, 26 Nov 2012 11:23:59 +0100
+Subject: [PATCH] Fix assertion fail at drawing selection of not yet fully
+ synchronized document
+
+2012-11-26  Armin Burgmeier  <armin@arbur.net>
+
+	* libinftextgtk/inf-text-gtk-view.c: Fix assertion fail at drawing selection
+	of not yet fully synchronized document.
+---
+ ChangeLog                         |    5 +++++
+ libinftextgtk/inf-text-gtk-view.c |   12 +++++++-----
+ 2 files changed, 12 insertions(+), 5 deletions(-)
+
+diff --git a/libinftextgtk/inf-text-gtk-view.c b/libinftextgtk/inf-text-gtk-view.c
+index d7917a9..10c51f1 100644
+--- a/libinftextgtk/inf-text-gtk-view.c
++++ b/libinftextgtk/inf-text-gtk-view.c
+@@ -971,11 +971,6 @@ inf_text_gtk_view_expose_event_after_cb(GtkWidget* widget,
+         if(sel > 0)
+         {
+           end = begin + sel;
+-          g_assert(
+-            end <= gtk_text_buffer_get_char_count(
+-              gtk_text_view_get_buffer(priv->textview)
+-            )
+-          );
+         }
+         else
+         {
+@@ -985,6 +980,13 @@ inf_text_gtk_view_expose_event_after_cb(GtkWidget* widget,
+           begin += sel;
+         }
+ 
++        /* This can happen if the document is not yet fully loaded, i.e. synchronization
++         * is still in progress. */
++        if(begin > gtk_text_buffer_get_char_count(gtk_text_view_get_buffer(priv->textview)))
++          begin = gtk_text_buffer_get_char_count(gtk_text_view_get_buffer(priv->textview));
++        if(end > gtk_text_buffer_get_char_count(gtk_text_view_get_buffer(priv->textview)))
++          end = gtk_text_buffer_get_char_count(gtk_text_view_get_buffer(priv->textview));
++
+         begin = MIN(MAX(begin, area_begin), area_end);
+         end = MIN(MAX(end, area_begin), area_end);
+         g_assert(end >= begin);
+-- 
+1.7.10.4
+
diff -Nru libinfinity-0.5.2/debian/patches/series libinfinity-0.5.2/debian/patches/series
--- libinfinity-0.5.2/debian/patches/series	2012-10-24 00:04:19.000000000 +0200
+++ libinfinity-0.5.2/debian/patches/series	2012-11-27 14:27:58.000000000 +0100
@@ -1,6 +1,7 @@
 0001-Check-for-libm-for-libinftextgtk.patch
 0001-Fix-a-few-enumeration-type-registration.patch
 0001-Fix-a-crash-when-a-connection-goes-down-while-attemp.patch
+0001-Fix-assertion-fail-at-drawing-selection.patch
 0001-Rework-Gtk3-support.patch
 0002-gtk3-test-Makefile.am.patch
 0003-gtk3-docs.patch

--- End Message ---
--- Begin Message ---
On Wed, 2012-11-28 at 20:45 +0100, Philipp Kern wrote:
> Please unblock package libinfinity
> 
> 0.5.2-6 fixes a crash of gobby-0.5 when the connection to the gobby server is
> slow and if some text is selected towards the end of the buffer. This caused
> crashes at both DebConf11 and at a recent BSP.

Unblocked; thanks.

Regards,

Adam

--- End Message ---

Reply to: