From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 13880 invoked by alias); 8 Apr 2011 14:18:41 -0000 Received: (qmail 13861 invoked by uid 9737); 8 Apr 2011 14:18:41 -0000 Date: Fri, 08 Apr 2011 14:18:00 -0000 Message-ID: <20110408141841.13859.qmail@sourceware.org> From: zkabelac@sourceware.org To: lvm-devel@redhat.com, lvm2-cvs@sourceware.org Subject: LVM2/daemons/dmeventd/plugins/snapshot dmevent ... Mailing-List: contact lvm2-cvs-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Post: List-Help: , Sender: lvm2-cvs-owner@sourceware.org X-SW-Source: 2011-04/txt/msg00008.txt.bz2 CVSROOT: /cvs/lvm2 Module name: LVM2 Changes by: zkabelac@sourceware.org 2011-04-08 14:18:40 Modified files: daemons/dmeventd/plugins/snapshot: dmeventd_snapshot.c Log message: Use dm_snprintf and fix differently signed comparation. Patches: http://sourceware.org/cgi-bin/cvsweb.cgi/LVM2/daemons/dmeventd/plugins/snapshot/dmeventd_snapshot.c.diff?cvsroot=lvm2&r1=1.14&r2=1.15 --- LVM2/daemons/dmeventd/plugins/snapshot/dmeventd_snapshot.c 2011/02/14 14:26:36 1.14 +++ LVM2/daemons/dmeventd/plugins/snapshot/dmeventd_snapshot.c 2011/04/08 14:18:40 1.15 @@ -125,8 +125,8 @@ syslog(LOG_ERR, "Unable to determine VG name from %s.", device); return 0; } - if (sizeof(cmd_str) <= snprintf(cmd_str, sizeof(cmd_str), - "lvextend --use-policies %s/%s", vg, lv)) { + if (dm_snprintf(cmd_str, sizeof(cmd_str), + "lvextend --use-policies %s/%s", vg, lv) < 0) { syslog(LOG_ERR, "Unable to form LVM command: Device name too long."); return 0; }