[libqb] fix qb_timespec_add_ms()

Angus Salkeld asalkeld at fedoraproject.org
Tue Jan 10 09:53:46 UTC 2012


commit 9bd8c740ca3372dd6d9a4ffd67c1bda8b7731cd0
Author: Angus Salkeld <asalkeld at redhat.com>
Date:   Tue Jan 10 20:23:55 2012 +1100

    fix qb_timespec_add_ms()
    
    Signed-off-by: Angus Salkeld <asalkeld at redhat.com>

 0001-LOG-fix-the-default-syslog-filter.patch |   31 --------------------------
 0001-UTIL-fix-qb_timespec_add_ms.patch       |   31 ++++++++++++++++++++++++++
 libqb.spec                                   |    9 ++++++-
 3 files changed, 39 insertions(+), 32 deletions(-)
---
diff --git a/0001-UTIL-fix-qb_timespec_add_ms.patch b/0001-UTIL-fix-qb_timespec_add_ms.patch
new file mode 100644
index 0000000..55c6b4f
--- /dev/null
+++ b/0001-UTIL-fix-qb_timespec_add_ms.patch
@@ -0,0 +1,31 @@
+From ef64fd1186979bae8eee2babb04d9b5b29d3099b Mon Sep 17 00:00:00 2001
+From: Angus Salkeld <asalkeld at redhat.com>
+Date: Mon, 9 Jan 2012 10:24:08 +1100
+Subject: [PATCH] UTIL: fix qb_timespec_add_ms()
+
+It was overwriting the timespec value instead of
+add to it.
+
+Signed-off-by: Angus Salkeld <asalkeld at redhat.com>
+---
+ lib/util.c |    4 ++--
+ 1 files changed, 2 insertions(+), 2 deletions(-)
+
+diff --git a/lib/util.c b/lib/util.c
+index 51c94bb..6b835cc 100644
+--- a/lib/util.c
++++ b/lib/util.c
+@@ -126,8 +126,8 @@ void
+ qb_timespec_add_ms(struct timespec *ts, int32_t ms)
+ {
+ #ifndef S_SPLINT_S
+-	ts->tv_sec = ms / 1000;
+-	ts->tv_nsec = (ms % 1000) * QB_TIME_NS_IN_MSEC;
++	ts->tv_sec += ms / 1000;
++	ts->tv_nsec += (ms % 1000) * QB_TIME_NS_IN_MSEC;
+ 	if (ts->tv_nsec >= 1000000000L) {
+ 		ts->tv_sec++;
+ 		ts->tv_nsec = ts->tv_nsec - 1000000000L;
+-- 
+1.7.7.5
+
diff --git a/libqb.spec b/libqb.spec
index 29d613f..631fa04 100644
--- a/libqb.spec
+++ b/libqb.spec
@@ -1,6 +1,6 @@
 Name:           libqb
 Version:        0.8.1
-Release:        1%{?dist}
+Release:        2%{?dist}
 Summary:        An IPC library for high performance servers
 
 Group:          System Environment/Libraries
@@ -9,6 +9,8 @@ URL:            http://www.libqb.org
 Source0:        https://fedorahosted.org/releases/q/u/quarterback/%{name}-%{version}.tar.xz
 BuildRoot:      %{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n)
 
+Patch1:         0001-UTIL-fix-qb_timespec_add_ms.patch
+
 BuildRequires:  libtool doxygen procps check-devel
 
 #Requires: <nothing>
@@ -20,6 +22,8 @@ Initially these are IPC and poll.
 %prep
 %setup -q
 
+%patch1 -p1
+
 %build
 %configure --disable-static
 make %{?_smp_mflags}
@@ -63,6 +67,9 @@ developing applications that use %{name}.
 %{_mandir}/man3/qb*3*
 
 %changelog
+* Tue Jan 10 2012  Angus Salkeld <asalkeld at redhat.com> - 0.8.1-2
+- fix qb_timespec_add_ms()
+
 * Thu Jan 5 2012  Angus Salkeld <asalkeld at redhat.com> - 0.8.1-1
 - Rebased to 0.8.1 (#771914)
 


More information about the scm-commits mailing list