vgritsenko    2004/02/18 18:50:44

  Modified:    java/src/org/apache/xindice/client/xmldb/embed
                        CollectionImpl.java
  Log:
  Add check on null/empty collection name
  
  Revision  Changes    Path
  1.32      +11 -4     
xml-xindice/java/src/org/apache/xindice/client/xmldb/embed/CollectionImpl.java
  
  Index: CollectionImpl.java
  ===================================================================
  RCS file: 
/home/cvs/xml-xindice/java/src/org/apache/xindice/client/xmldb/embed/CollectionImpl.java,v
  retrieving revision 1.31
  retrieving revision 1.32
  diff -u -r1.31 -r1.32
  --- CollectionImpl.java       8 Feb 2004 02:43:01 -0000       1.31
  +++ CollectionImpl.java       19 Feb 2004 02:50:44 -0000      1.32
  @@ -502,13 +502,20 @@
   
       /* see superclass for documentation */
       public void removeCollection(String childName) throws XMLDBException {
  +
  +        if (null == childName || childName.length() == 0) {
  +            throw new XMLDBException(ErrorCodes.NO_SUCH_COLLECTION,
  +                                     FaultCodes.COL_COLLECTION_NOT_FOUND,
  +                                     "Cannot remove child collection '" + 
childName + "': Name is empty");
  +        }
  +
           checkOpen();
           try {
               col.dropCollection(col.getCollection(childName));
           } catch (Exception e) {
               throw 
FaultCodes.createXMLDBException(ErrorCodes.INVALID_COLLECTION,
                                                     FaultCodes.GEN_UNKNOWN,
  -                                                  "Cannot remove child 
collection", e);
  +                                                  "Cannot remove child 
collection '" + childName + "'", e);
           }
       }
   
  @@ -553,7 +560,7 @@
   
       // search result handling
   
  -    private Document nodesetToDocument(NodeSet ns) {
  +    private static Document nodesetToDocument(NodeSet ns) {
   
           //
           // FIXME: There is a copy of this same code in 
org.apache.xindice.server.xmldb.rpc.messages.Query#queryWrapper
  
  
  

Reply via email to