From b36e4dfb8df1f0712c0fb6aed490fa6d645c74bc Mon Sep 17 00:00:00 2001 From: "Christian P. MOMON" Date: Fri, 13 Aug 2010 10:33:10 +0200 Subject: [PATCH] Cleaning. --- .../kiss4web/SimpleServletDispatcher.java | 2 - src/fr/devinsy/kiss4web/security/Groups.java | 2 +- src/fr/devinsy/util/Digester.java | 5 +- src/fr/devinsy/util/FileIteratorState.java | 1 - .../devinsy/util/cmdexec/StreamGobbler.java | 47 +++++++++++++++---- src/fr/devinsy/util/cmdexec/Wrapper.java | 11 ++--- src/fr/devinsy/util/unix/EtcGroupFile.java | 18 ++----- .../devinsy/util/unix/EtcGroupFileReader.java | 5 +- .../util/unix/EtcPasswdFileReader.java | 5 +- src/fr/devinsy/util/unix/Group.java | 9 +--- src/fr/devinsy/util/unix/Groups.java | 14 ++---- src/fr/devinsy/util/unix/Unix.java | 6 +-- src/fr/devinsy/util/unix/User.java | 31 ++++++++---- src/fr/devinsy/util/unix/Users.java | 14 ++---- src/fr/devinsy/util/unix/acl/Acl.java | 3 +- src/fr/devinsy/util/unix/acl/AclEntries.java | 4 +- src/fr/devinsy/util/unix/acl/AclEntry.java | 2 +- .../devinsy/util/web/SimpleSecurityAgent.java | 9 ++-- 18 files changed, 99 insertions(+), 89 deletions(-) diff --git a/src/fr/devinsy/kiss4web/SimpleServletDispatcher.java b/src/fr/devinsy/kiss4web/SimpleServletDispatcher.java index efbdfef..f5af870 100644 --- a/src/fr/devinsy/kiss4web/SimpleServletDispatcher.java +++ b/src/fr/devinsy/kiss4web/SimpleServletDispatcher.java @@ -18,8 +18,6 @@ import javax.servlet.http.HttpServlet; import javax.servlet.http.HttpServletRequest; import javax.servlet.http.HttpServletResponse; - - import fr.devinsy.util.StringConcatenator; diff --git a/src/fr/devinsy/kiss4web/security/Groups.java b/src/fr/devinsy/kiss4web/security/Groups.java index 350363d..1cfdfd1 100644 --- a/src/fr/devinsy/kiss4web/security/Groups.java +++ b/src/fr/devinsy/kiss4web/security/Groups.java @@ -106,7 +106,7 @@ public class Groups extends Vector /** - * + * */ public String getLoginGroupsString (String login) { diff --git a/src/fr/devinsy/util/Digester.java b/src/fr/devinsy/util/Digester.java index 6541bea..066b460 100644 --- a/src/fr/devinsy/util/Digester.java +++ b/src/fr/devinsy/util/Digester.java @@ -6,9 +6,10 @@ */ package fr.devinsy.util; -import java.io.*; import java.io.File; -import java.security.*; +import java.io.FileInputStream; +import java.io.InputStream; +import java.security.MessageDigest; /** diff --git a/src/fr/devinsy/util/FileIteratorState.java b/src/fr/devinsy/util/FileIteratorState.java index 58f9b9f..eed4946 100644 --- a/src/fr/devinsy/util/FileIteratorState.java +++ b/src/fr/devinsy/util/FileIteratorState.java @@ -2,7 +2,6 @@ package fr.devinsy.util; import java.io.File; import java.util.Iterator; -import java.util.regex.Pattern; /** diff --git a/src/fr/devinsy/util/cmdexec/StreamGobbler.java b/src/fr/devinsy/util/cmdexec/StreamGobbler.java index 37b9af6..dea3226 100644 --- a/src/fr/devinsy/util/cmdexec/StreamGobbler.java +++ b/src/fr/devinsy/util/cmdexec/StreamGobbler.java @@ -1,6 +1,9 @@ package fr.devinsy.util.cmdexec; -import java.io.*; +import java.io.BufferedReader; +import java.io.IOException; +import java.io.InputStream; +import java.io.InputStreamReader; /** @@ -21,7 +24,9 @@ public class StreamGobbler extends Thread protected boolean isOverStatus; // Important if the caller wants have complete stream in case of very short command. - // + /** + * + */ StreamGobbler () { this.type = ""; @@ -31,7 +36,11 @@ public class StreamGobbler extends Thread } - // + /** + * + * @param type + * @param streamWay + */ StreamGobbler (String type, int streamWay) { this.type = type; @@ -41,7 +50,11 @@ public class StreamGobbler extends Thread } - // + /** + * + * @param is + * @param type + */ StreamGobbler (InputStream is, String type) { this.is = is; @@ -52,7 +65,12 @@ public class StreamGobbler extends Thread } - // + /** + * + * @param is + * @param type + * @param streamWay + */ StreamGobbler (InputStream is, String type, int streamWay) { this.is = is; @@ -63,14 +81,19 @@ public class StreamGobbler extends Thread } - // + /** + * + * @param is + */ public void setInputStream (InputStream is) { this.is = is; } - // + /** + * + */ public void run () { try @@ -110,7 +133,10 @@ public class StreamGobbler extends Thread } - // + /** + * + * @return + */ public String getStream () { String result; @@ -129,7 +155,10 @@ public class StreamGobbler extends Thread } - // + /** + * + * @return + */ public boolean isOver () { boolean result; diff --git a/src/fr/devinsy/util/cmdexec/Wrapper.java b/src/fr/devinsy/util/cmdexec/Wrapper.java index 9e4ce55..7886a0f 100644 --- a/src/fr/devinsy/util/cmdexec/Wrapper.java +++ b/src/fr/devinsy/util/cmdexec/Wrapper.java @@ -1,6 +1,9 @@ package fr.devinsy.util.cmdexec; -import java.io.*; +import java.io.BufferedReader; +import java.io.FileReader; +import java.io.IOException; +import java.io.PrintWriter; /** @@ -9,12 +12,8 @@ import java.io.*; public class Wrapper { static private final Wrapper instance = new Wrapper (); - static private org.apache.log4j.Logger logger; + static private org.apache.log4j.Logger logger = org.apache.log4j.Logger.getLogger (Wrapper.class); - static - { - logger = org.apache.log4j.Logger.getLogger (Wrapper.class); - } /** * return instance of the controller diff --git a/src/fr/devinsy/util/unix/EtcGroupFile.java b/src/fr/devinsy/util/unix/EtcGroupFile.java index 7dae46c..688a930 100644 --- a/src/fr/devinsy/util/unix/EtcGroupFile.java +++ b/src/fr/devinsy/util/unix/EtcGroupFile.java @@ -1,7 +1,6 @@ package fr.devinsy.util.unix; - -import java.util.*; +import java.util.Vector; /** @@ -9,18 +8,9 @@ import java.util.*; */ public class EtcGroupFile extends CachedFile { - static private org.apache.log4j.Logger logger; - static - { - logger = org.apache.log4j.Logger.getLogger (EtcGroupFile.class); - } - - static EtcGroupFile instance; - static - { - instance = null; - } + //static private org.apache.log4j.Logger logger = org.apache.log4j.Logger.getLogger (EtcGroupFile.class); + static EtcGroupFile instance = null; protected Groups groups; /** @@ -164,6 +154,7 @@ public class EtcGroupFile extends CachedFile return (result); } + /** * */ @@ -180,7 +171,6 @@ public class EtcGroupFile extends CachedFile } - /** * */ diff --git a/src/fr/devinsy/util/unix/EtcGroupFileReader.java b/src/fr/devinsy/util/unix/EtcGroupFileReader.java index 4e3cd39..e213f97 100644 --- a/src/fr/devinsy/util/unix/EtcGroupFileReader.java +++ b/src/fr/devinsy/util/unix/EtcGroupFileReader.java @@ -1,7 +1,8 @@ package fr.devinsy.util.unix; -import java.io.*; - +import java.io.BufferedReader; +import java.io.FileNotFoundException; +import java.io.FileReader; /** diff --git a/src/fr/devinsy/util/unix/EtcPasswdFileReader.java b/src/fr/devinsy/util/unix/EtcPasswdFileReader.java index 1e09f7d..0390c97 100644 --- a/src/fr/devinsy/util/unix/EtcPasswdFileReader.java +++ b/src/fr/devinsy/util/unix/EtcPasswdFileReader.java @@ -1,7 +1,8 @@ package fr.devinsy.util.unix; -import java.io.*; - +import java.io.BufferedReader; +import java.io.FileNotFoundException; +import java.io.FileReader; /** diff --git a/src/fr/devinsy/util/unix/Group.java b/src/fr/devinsy/util/unix/Group.java index 5973730..fef9b86 100644 --- a/src/fr/devinsy/util/unix/Group.java +++ b/src/fr/devinsy/util/unix/Group.java @@ -1,7 +1,6 @@ package fr.devinsy.util.unix; - -import java.util.*; +import java.util.Vector; /** @@ -9,12 +8,8 @@ import java.util.*; */ public class Group { - static private org.apache.log4j.Logger logger; + //static private org.apache.log4j.Logger logger = org.apache.log4j.Logger.getLogger (Group.class); - static - { - logger = org.apache.log4j.Logger.getLogger (Group.class); - } /* * /usr/include/grp.h diff --git a/src/fr/devinsy/util/unix/Groups.java b/src/fr/devinsy/util/unix/Groups.java index 2b83bd2..12c4e5e 100644 --- a/src/fr/devinsy/util/unix/Groups.java +++ b/src/fr/devinsy/util/unix/Groups.java @@ -1,7 +1,7 @@ package fr.devinsy.util.unix; - -import java.util.*; +import java.util.Iterator; +import java.util.Vector; /** @@ -11,12 +11,7 @@ public class Groups extends Vector { private static final long serialVersionUID = 5802487312198869603L; - static private org.apache.log4j.Logger logger; - - static - { - logger = org.apache.log4j.Logger.getLogger (Groups.class); - } + //static private org.apache.log4j.Logger logger = org.apache.log4j.Logger.getLogger (Groups.class); /** @@ -206,8 +201,7 @@ public class Groups extends Vector { String result; - StringBuffer out; - out = new StringBuffer (); + StringBuffer out = new StringBuffer (); Iterator iterator = this.iterator (); diff --git a/src/fr/devinsy/util/unix/Unix.java b/src/fr/devinsy/util/unix/Unix.java index 9872da0..11a1bd7 100644 --- a/src/fr/devinsy/util/unix/Unix.java +++ b/src/fr/devinsy/util/unix/Unix.java @@ -1,17 +1,14 @@ package fr.devinsy.util.unix; - import java.io.File; import java.util.Vector; import fr.devinsy.util.StringConcatenator; import fr.devinsy.util.cmdexec.CmdExec; -import fr.devinsy.util.cmdexec.StreamGobbler; import fr.devinsy.util.unix.acl.Acl; import fr.devinsy.util.unix.acl.AclManager; - /** * */ @@ -423,6 +420,7 @@ public class Unix } } + /** * */ @@ -604,4 +602,4 @@ public class Unix } } - // //////////////////////////////////////////////////////////////////////// \ No newline at end of file +// //////////////////////////////////////////////////////////////////////// \ No newline at end of file diff --git a/src/fr/devinsy/util/unix/User.java b/src/fr/devinsy/util/unix/User.java index a8fd43d..8830621 100644 --- a/src/fr/devinsy/util/unix/User.java +++ b/src/fr/devinsy/util/unix/User.java @@ -1,19 +1,12 @@ package fr.devinsy.util.unix; - - /** * */ public class User { - static private org.apache.log4j.Logger logger; - - static - { - logger = org.apache.log4j.Logger.getLogger (User.class); - } + //static private org.apache.log4j.Logger logger = org.apache.log4j.Logger.getLogger (User.class); /* * /usr/include/pwd.h @@ -83,6 +76,7 @@ public class User return (result); } + /** * */ @@ -95,6 +89,7 @@ public class User // return (result); } + /** * @@ -108,6 +103,7 @@ public class User // return (result); } + /** * @@ -122,6 +118,7 @@ public class User return (result); } + /** * */ @@ -135,6 +132,7 @@ public class User return (result); } + /** * */ @@ -148,6 +146,7 @@ public class User return (result); } + /** * */ @@ -161,6 +160,7 @@ public class User return (result); } + /** * */ @@ -174,6 +174,7 @@ public class User return (result); } + /** * */ @@ -187,6 +188,7 @@ public class User return (result); } + /** * */ @@ -200,6 +202,7 @@ public class User return (result); } + /** * */ @@ -213,6 +216,7 @@ public class User return (result); } + /** * */ @@ -225,6 +229,7 @@ public class User // return (result); } + /** * @@ -239,6 +244,7 @@ public class User return (result); } + /** * */ @@ -265,6 +271,7 @@ public class User return (result); } + /** * */ @@ -273,6 +280,7 @@ public class User this.homeDirectory = homeDirectory; } + /** * */ @@ -281,6 +289,7 @@ public class User this.gid = gid; } + /** * */ @@ -289,6 +298,7 @@ public class User this.login = login; } + /** * */ @@ -297,6 +307,7 @@ public class User this.password = password; } + /** * */ @@ -305,6 +316,7 @@ public class User this.password = password; } + /** * */ @@ -313,6 +325,7 @@ public class User this.realName = realName; } + /** * */ @@ -321,6 +334,7 @@ public class User this.shell = shell; } + /** * */ @@ -328,6 +342,7 @@ public class User { this.uid = uid; } + /** * diff --git a/src/fr/devinsy/util/unix/Users.java b/src/fr/devinsy/util/unix/Users.java index 4af0c37..83fecbc 100644 --- a/src/fr/devinsy/util/unix/Users.java +++ b/src/fr/devinsy/util/unix/Users.java @@ -1,7 +1,7 @@ package fr.devinsy.util.unix; - -import java.util.*; +import java.util.Iterator; +import java.util.Vector; /** @@ -10,12 +10,7 @@ import java.util.*; public class Users extends Vector { private static final long serialVersionUID = -7178304512851592399L; - static private org.apache.log4j.Logger logger; - - static - { - logger = org.apache.log4j.Logger.getLogger (Users.class); - } + //static private org.apache.log4j.Logger logger = org.apache.log4j.Logger.getLogger (Users.class); /** @@ -142,9 +137,6 @@ public class Users extends Vector } - - - /** * */ diff --git a/src/fr/devinsy/util/unix/acl/Acl.java b/src/fr/devinsy/util/unix/acl/Acl.java index e6e5b31..0959d1d 100644 --- a/src/fr/devinsy/util/unix/acl/Acl.java +++ b/src/fr/devinsy/util/unix/acl/Acl.java @@ -6,7 +6,7 @@ package fr.devinsy.util.unix.acl; */ public class Acl { - static private org.apache.log4j.Logger logger = org.apache.log4j.Logger.getLogger (Acl.class); + // static private org.apache.log4j.Logger logger = org.apache.log4j.Logger.getLogger (Acl.class); /* @@ -36,6 +36,7 @@ public class Acl protected AclEntries currentAcl; protected AclEntries defaultAcl; + /** * */ diff --git a/src/fr/devinsy/util/unix/acl/AclEntries.java b/src/fr/devinsy/util/unix/acl/AclEntries.java index dad87d7..abd2677 100644 --- a/src/fr/devinsy/util/unix/acl/AclEntries.java +++ b/src/fr/devinsy/util/unix/acl/AclEntries.java @@ -1,6 +1,5 @@ package fr.devinsy.util.unix.acl; - import java.util.Iterator; import java.util.Vector; @@ -11,7 +10,7 @@ import java.util.Vector; public class AclEntries extends Vector { private static final long serialVersionUID = 5802487312198869603L; - static private org.apache.log4j.Logger logger = org.apache.log4j.Logger.getLogger (AclEntries.class); + //static private org.apache.log4j.Logger logger = org.apache.log4j.Logger.getLogger (AclEntries.class); /** * @@ -176,6 +175,7 @@ public class AclEntries extends Vector return(result); } + /** * */ diff --git a/src/fr/devinsy/util/unix/acl/AclEntry.java b/src/fr/devinsy/util/unix/acl/AclEntry.java index 9c1abbd..5e53625 100644 --- a/src/fr/devinsy/util/unix/acl/AclEntry.java +++ b/src/fr/devinsy/util/unix/acl/AclEntry.java @@ -6,7 +6,7 @@ package fr.devinsy.util.unix.acl; */ public class AclEntry { - static private org.apache.log4j.Logger logger = org.apache.log4j.Logger.getLogger (AclEntry.class); + //static private org.apache.log4j.Logger logger = org.apache.log4j.Logger.getLogger (AclEntry.class); /* diff --git a/src/fr/devinsy/util/web/SimpleSecurityAgent.java b/src/fr/devinsy/util/web/SimpleSecurityAgent.java index 898035a..bfb767d 100644 --- a/src/fr/devinsy/util/web/SimpleSecurityAgent.java +++ b/src/fr/devinsy/util/web/SimpleSecurityAgent.java @@ -1,11 +1,8 @@ package fr.devinsy.util.web; - -import java.security.MessageDigest; -import java.security.NoSuchAlgorithmException; - -import javax.servlet.http.*; -import sun.security.provider.MD5; +import javax.servlet.http.Cookie; +import javax.servlet.http.HttpServletRequest; +import javax.servlet.http.HttpServletResponse; /**