Attachment 592615 Details for Bug 833007 – [PATCH 1/4] added error checks and avoid access to the 'None' cds

[patch] [PATCH 1/4] added error checks and avoid access to the 'None' cds

0001-added-error-checks-and-avoid-access-to-the-None-cds.patch (text/plain), 1.14 KB, created by Satoru SATOH on 2012-06-18 11:43:17 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Satoru SATOH

Created: 2012-06-18 11:43:17 UTC

Size: 1.14 KB

patch

obsolete

>From 1981e834c3aded3693a790d333ed94b6c0a0e39b Mon Sep 17 00:00:00 2001
>From: Satoru SATOH <ssato@redhat.com>
>Date: Mon, 18 Jun 2012 19:15:29 +0900
>Subject: [PATCH 1/4] added error checks and avoid access to the 'None' cds
>
>---
> etc/rhui/client-yum-plugin/rhui-lb.py |    8 ++++++++
> 1 file changed, 8 insertions(+)
>
>diff --git a/etc/rhui/client-yum-plugin/rhui-lb.py b/etc/rhui/client-yum-plugin/rhui-lb.py
>index 13f86c8..e4e8394 100644
>--- a/etc/rhui/client-yum-plugin/rhui-lb.py
>+++ b/etc/rhui/client-yum-plugin/rhui-lb.py
>@@ -47,6 +47,10 @@ def postreposetup_hook(conduit):
>     cds_balancers = f.read().split()
>     f.close()
> 
>+    if not cds_balancers:
>+        conduit.error(0, 'No CDS load balancers found in the lists.')
>+        return
>+
>     # Figure out which load balancer is up
>     up_lb = None
>     for lb in cds_balancers:
>@@ -83,6 +87,10 @@ def postreposetup_hook(conduit):
>         except:
>             pass
> 
>+    if up_lb is None:
>+        conduit.error(0, 'No accessible CDS load balancers found.')
>+        return
>+
>     for repo in rhui_repos:
>         # Yank out the original mirror list entry and stuff in
>         # the good one
>-- 
>1.7.10.2
>

Actions: View | Diff

Attachments on bug 833007: 592615 | 592619 | 592621 | 592625