Attachment 640478 Details for Bug 874278 – Patch proposed

[patch] Patch proposed

0001-errata.findByCve.patch (text/plain), 1.49 KB, created by Marcelo Moreira de Mello on 2012-11-07 23:56:21 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Marcelo Moreira de Mello

Created: 2012-11-07 23:56:21 UTC

Size: 1.49 KB

patch

obsolete

>From: Marcelo Moreira de Mello <mmello@redhat.com>
>Date: Wed, 7 Nov 2012 21:47:20 -0200
>Subject: [PATCH] fixed errata.findByCve() method for seek big lists
>
>Caused by: java.util.ConcurrentModificationException     
>  at java.util.LinkedList$LinkIterator.next(LinkedList.java:119)
>  at com.redhat.rhn.frontend.xmlrpc.errata.ErrataHandler.findByCve(ErrataHandler.java:1427)
>
>---
> .../src/com/redhat/rhn/frontend/xmlrpc/errata/ErrataHandler.java     | 5 +++--
> 1 file changed, 3 insertions(+), 2 deletions(-)
>
>diff --git a/java/code/src/com/redhat/rhn/frontend/xmlrpc/errata/ErrataHandler.java b/java/code/src/com/redhat/rhn/frontend/xmlrpc/errata/ErrataHandler.java
>index f86058c..2a7baaa 100644
>--- a/java/code/src/com/redhat/rhn/frontend/xmlrpc/errata/ErrataHandler.java
>+++ b/java/code/src/com/redhat/rhn/frontend/xmlrpc/errata/ErrataHandler.java
>@@ -1461,11 +1461,12 @@ public class ErrataHandler extends BaseHandler {
>         User loggedInUser = getLoggedInUser(sessionKey);
> 
>         List<Errata> erratas = ErrataManager.lookupByCVE(cveName);
>-        for (Errata errata : erratas) {
>+        for (Iterator iter = erratas.iterator(); iter.hasNext();) {
>+            Errata errata = (Errata) iter.next();
>             // Remove errata that do not apply to the user's org
>             if (errata.getOrg() != null &&
>                     !errata.getOrg().equals(loggedInUser.getOrg())) {
>-                erratas.remove(errata);
>+                iter.remove();
>             }
>         }
>         return erratas;
>-- 
>1.8.0
>

Actions: View | Diff

Attachments on bug 874278: 640478