From owner-freebsd-ports-bugs@FreeBSD.ORG Fri Oct 23 12:20:02 2009 Return-Path: Delivered-To: freebsd-ports-bugs@hub.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 3FEC71065692 for ; Fri, 23 Oct 2009 12:20:02 +0000 (UTC) (envelope-from gnats@FreeBSD.org) Received: from freefall.freebsd.org (freefall.freebsd.org [IPv6:2001:4f8:fff6::28]) by mx1.freebsd.org (Postfix) with ESMTP id F1DB88FC12 for ; Fri, 23 Oct 2009 12:20:01 +0000 (UTC) Received: from freefall.freebsd.org (localhost [127.0.0.1]) by freefall.freebsd.org (8.14.3/8.14.3) with ESMTP id n9NCK1Nj038050 for ; Fri, 23 Oct 2009 12:20:01 GMT (envelope-from gnats@freefall.freebsd.org) Received: (from gnats@localhost) by freefall.freebsd.org (8.14.3/8.14.3/Submit) id n9NCK1jA038049; Fri, 23 Oct 2009 12:20:01 GMT (envelope-from gnats) Resent-Date: Fri, 23 Oct 2009 12:20:01 GMT Resent-Message-Id: <200910231220.n9NCK1jA038049@freefall.freebsd.org> Resent-From: FreeBSD-gnats-submit@FreeBSD.org (GNATS Filer) Resent-To: freebsd-ports-bugs@FreeBSD.org Resent-Reply-To: FreeBSD-gnats-submit@FreeBSD.org, Nikos Ntarmos Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 4909D106568D for ; Fri, 23 Oct 2009 12:13:49 +0000 (UTC) (envelope-from ntarmos@cs.uoi.gr) Received: from gaia.cs.uoi.gr (gaia.cs.uoi.gr [195.130.121.201]) by mx1.freebsd.org (Postfix) with ESMTP id BDFB98FC18 for ; Fri, 23 Oct 2009 12:13:48 +0000 (UTC) Received: from zeus.cs.uoi.gr (zeus.cs.uoi.gr [195.130.121.11]) by gaia.cs.uoi.gr (8.14.1/8.14.1) with ESMTP id n9NBnRw9046719 for ; Fri, 23 Oct 2009 14:49:32 +0300 (EEST) (envelope-from ntarmos@cs.uoi.gr) Received: from zeus.cs.uoi.gr (localhost [127.0.0.1]) by zeus.cs.uoi.gr (8.13.5/8.13.5) with ESMTP id n9NBnM89010013 for ; Fri, 23 Oct 2009 14:49:27 +0300 (EEST) Received: (from ntarmos@localhost) by zeus.cs.uoi.gr (8.13.5/8.13.5/Submit) id n9NBnMXS010012; Fri, 23 Oct 2009 14:49:22 +0300 (EEST) Message-Id: <200910231149.n9NBnMXS010012@zeus.cs.uoi.gr> Date: Fri, 23 Oct 2009 14:49:22 +0300 (EEST) From: Nikos Ntarmos To: FreeBSD-gnats-submit@FreeBSD.org X-Send-Pr-Version: 3.113 Cc: Subject: ports/139863: [maintainer-update] x11/xcb-util: division by zero bugfix X-BeenThere: freebsd-ports-bugs@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list Reply-To: Nikos Ntarmos List-Id: Ports bug reports List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 23 Oct 2009 12:20:02 -0000 >Number: 139863 >Category: ports >Synopsis: [maintainer-update] x11/xcb-util: division by zero bugfix >Confidential: no >Severity: non-critical >Priority: low >Responsible: freebsd-ports-bugs >State: open >Quarter: >Keywords: >Date-Required: >Class: maintainer-update >Submitter-Id: current-users >Arrival-Date: Fri Oct 23 12:20:01 UTC 2009 >Closed-Date: >Last-Modified: >Originator: Nikos Ntarmos >Release: FreeBSD 7.2-RELEASE-p4 i386 >Organization: Computer Science Dept., U. of Ioannina, Greece >Environment: System: FreeBSD ace.cs.uoi.gr 7.2-RELEASE-p4 FreeBSD 7.2-RELEASE-p4 #2: Sat Oct 3 01:59:15 EEST 2009 ntarmos@ace.cs.uoi.gr:/opt/obj/opt/src/sys/ACE i386 >Description: Ashish Shukla (wahjava@gmail.com) reported a crash with x11-wm/awesome on amd64. With his help, we narrowed it down to a division by zero going unnoticed in xcb-util. This is now fixed upstream but there won't be a new release any time soon, so I'm adding the relevant patch to the port. >How-To-Repeat: >Fix: --- xcb-util-0.3.6_1.diff begins here --- --- Makefile.orig +++ Makefile @@ -7,6 +7,7 @@ PORTNAME= xcb-util PORTVERSION= 0.3.6 +PORTREVISION= 1 CATEGORIES= x11 MASTER_SITES= http://xcb.freedesktop.org/dist/ --- files/patch-icccm-icccm.c.orig +++ files/patch-icccm-icccm.c @@ -0,0 +1,20 @@ +--- icccm/icccm.c.orig 2009-08-24 11:47:36.000000000 +0300 ++++ icccm/icccm.c 2009-10-21 21:01:52.000000000 +0300 +@@ -627,15 +627,13 @@ + xcb_get_wm_hints_from_reply(xcb_wm_hints_t *hints, + xcb_get_property_reply_t *reply) + { +- if(!reply) ++ if(!reply || reply->type != WM_HINTS || reply->format != 32) + return 0; + + int length = xcb_get_property_value_length(reply); + int num_elem = length / (reply->format / 8); + +- if (reply->type != WM_HINTS +- || reply->format != 32 +- || num_elem < XCB_NUM_WM_HINTS_ELEMENTS - 1) ++ if(num_elem < XCB_NUM_WM_HINTS_ELEMENTS - 1) + return 0; + + memcpy(hints, (xcb_size_hints_t *) xcb_get_property_value(reply), length); --- xcb-util-0.3.6_1.diff ends here --- >Release-Note: >Audit-Trail: >Unformatted: