Attachment 192457 Details for Bug 341613 – backport patch

Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

[patch] backport patch

patch.txt (text/plain), 1.71 KB, created by David McKnight

on 2011-04-04 09:56:39 EDT

(hide)

Description:

Filename:

MIME Type:

Creator: David McKnight

Created: 2011-04-04 09:56:39 EDT

Size: 1.71 KB

patch

obsolete

>### Eclipse Workspace Patch 1.0
>#P org.eclipse.dstore.core
>Index: src/org/eclipse/dstore/core/server/ServerLogger.java
>===================================================================
>RCS file: /cvsroot/tools/org.eclipse.tm.rse/plugins/org.eclipse.dstore.core/src/org/eclipse/dstore/core/server/ServerLogger.java,v
>retrieving revision 1.16
>diff -u -r1.16 ServerLogger.java
>--- src/org/eclipse/dstore/core/server/ServerLogger.java	19 Mar 2010 23:20:27 -0000	1.16
>+++ src/org/eclipse/dstore/core/server/ServerLogger.java	4 Apr 2011 13:57:09 -0000
>@@ -1,5 +1,5 @@
> /********************************************************************************
>- * Copyright (c) 2002, 2010 IBM Corporation. All rights reserved.
>+ * Copyright (c) 2002, 2011 IBM Corporation. All rights reserved.
>  * This program and the accompanying materials are made available under the terms
>  * of the Eclipse Public License v1.0 which accompanies this distribution, and is
>  * available at http://www.eclipse.org/legal/epl-v10.html
>@@ -21,6 +21,7 @@
>  * David McKnight  (IBM)  - [269908] [dstore] rsecomm.log file management
>  * David McKnight  (IBM)  - [284787] [dstore] ability to disable RSECOMM_LOGFILE_MAX option
>  * David McKnight  (IBM)  - [305272] [dstore][multithread] log close in ServerLogger
>+ * Noriaki Takatsu (IBM)  - [341578] [dstore] ServerLogger is looped when IOError happens
>  ********************************************************************************/
> 
> package org.eclipse.dstore.core.server;
>@@ -142,7 +143,9 @@
> 					logFile.createNewFile();
> 					found = true;
> 				}
>-				catch (IOException e){}
>+				catch (IOException e){
>+					return null;
>+				}
> 			}
> 			else {
> 				// if the file exists, check it's size

Actions: View | Diff

Attachments on bug 341613: 192457