amoralej / rpms / 389-ds-base

Forked from rpms/389-ds-base 5 years ago
Clone

Blame SOURCES/0024-Ticket-47387-improve-logconv.pl-performance-with-lar.patch

ba46c7
From ecd128536dcec395876936a8d3abf5a1eea46089 Mon Sep 17 00:00:00 2001
ba46c7
From: Rich Megginson <rmeggins@redhat.com>
ba46c7
Date: Tue, 2 Jul 2013 14:38:49 -0600
ba46c7
Subject: [PATCH 24/28] Ticket #47387 - improve logconv.pl performance with large access logs
ba46c7
ba46c7
https://fedorahosted.org/389/ticket/47387
ba46c7
Reviewed by: mreynolds (Thanks!)
ba46c7
Branch: master
ba46c7
Fix Description: Fix some additional uninitialized variable warnings
ba46c7
Platforms tested: RHEL6 x86_64
ba46c7
Flag Day: no
ba46c7
Doc impact: no
ba46c7
(cherry picked from commit 8a23f5e0af41291d28c4ce7251d9bb2e94a5994c)
ba46c7
(cherry picked from commit 7a107bdb67dad58728977b3f410df1d94c6cdb91)
ba46c7
---
ba46c7
 ldap/admin/src/logconv.pl |    8 ++++----
ba46c7
 1 files changed, 4 insertions(+), 4 deletions(-)
ba46c7
ba46c7
diff --git a/ldap/admin/src/logconv.pl b/ldap/admin/src/logconv.pl
ba46c7
index b628d03..ec0fdf5 100755
ba46c7
--- a/ldap/admin/src/logconv.pl
ba46c7
+++ b/ldap/admin/src/logconv.pl
ba46c7
@@ -673,7 +673,7 @@ if ($verb eq "yes" || $usage =~ /u/){
ba46c7
 
ba46c7
 		my $notesCount = "1";
ba46c7
 		my $unindexedIp;
ba46c7
-		for (my $n = 0; $n <= scalar(@{$notesEtime}); $n++){
ba46c7
+		for (my $n = 0; $n < scalar(@{$notesEtime}); $n++){
ba46c7
 			if($conn_hash->{$notesConn->[$n]} eq ""){
ba46c7
 				$unindexedIp = "?";
ba46c7
 			} else {
ba46c7
@@ -726,7 +726,7 @@ if ($verb eq "yes" || $usage =~ /u/){
ba46c7
 
ba46c7
 		my $notesCount = "1";
ba46c7
 		my $unindexedIp;
ba46c7
-		for (my $n = 0; $n <= scalar(@{$notesEtime}); $n++){
ba46c7
+		for (my $n = 0; $n < scalar(@{$notesEtime}); $n++){
ba46c7
 			if($conn_hash->{$notesConn->[$n]} eq ""){
ba46c7
 				$unindexedIp = "?";
ba46c7
 			} else {
ba46c7
@@ -1821,7 +1821,7 @@ sub parseLineNormal
ba46c7
 		        $con = $1;
ba46c7
 		        if ($_ =~ /op= *([0-9]+)/i){ $op = $1;}
ba46c7
 		}
ba46c7
-		for (my $i=0; $i <= $vlvCount;$i++){
ba46c7
+		for (my $i=0; $i < $vlvCount;$i++){
ba46c7
 		        if ($vlvconn[$i] eq $con && $vlvop[$i] eq $op){ $vlvNotesACount++; $isVlvNotes="1";}
ba46c7
 		}
ba46c7
 		if($isVlvNotes == 0){
ba46c7
@@ -1846,7 +1846,7 @@ sub parseLineNormal
ba46c7
 		        $con = $1;
ba46c7
 		        if ($_ =~ /op= *([0-9]+)/i){ $op = $1;}
ba46c7
 		}
ba46c7
-		for (my $i=0; $i <= $vlvCount;$i++){
ba46c7
+		for (my $i=0; $i < $vlvCount;$i++){
ba46c7
 		        if ($vlvconn[$i] eq $con && $vlvop[$i] eq $op){ $vlvNotesUCount++; $isVlvNotes="1";}
ba46c7
 		}
ba46c7
 		if($isVlvNotes == 0){
ba46c7
-- 
ba46c7
1.7.1
ba46c7