[ignore] port rev 11291 (bugfix: InternalXMLSerializer must close str…

…eam on endDocument())

port rev 11292 (bugfix: move temp file creation to startDocument(), IOException now throw as SAXException)

svn path=/releases/eXist-1.4/; revision=11706

src/org/exist/xmldb/RemoteXMLResource.java

@@ -323,33 +323,45 @@ private class InternalXMLSerializer
     	extends SAXSerializer
     {
     	File tmpfile=null;
+    	OutputStreamWriter writer = null;
 
 		public InternalXMLSerializer() {
 			super();
+		}
+
+		public void startDocument() throws SAXException {
 			try {
 		    	File tmpfile=File.createTempFile("eXistRXR", ".xml");
 		    	tmpfile.deleteOnExit();
 		    	FileOutputStream fos=new FileOutputStream(tmpfile);
 		    	BufferedOutputStream bos=new BufferedOutputStream(fos);
-		    	OutputStreamWriter writer=new OutputStreamWriter(bos,"UTF-8");
+		    	writer=new OutputStreamWriter(bos,"UTF-8");
 				setOutput(writer, emptyProperties);
 			} catch(IOException ioe) {
-				// Don't know where to report!!!
+				throw new SAXException("Unable to create temp file for serialization data",ioe);
 			}
+
+			super.startDocument();
 		}
 
 		/**
 		 * @see org.xml.sax.DocumentHandler#endDocument()
 		 */
-		public void endDocument()
-			throws SAXException
+		public void endDocument() throws SAXException
 		{
 		    super.endDocument();
 		    try {
 		    	setContent(tmpfile);
 		    } catch(XMLDBException xe) {
 		    	throw new SAXException("Unable to close temp file containing serialized data",xe);
 		    }
+
+		    try {
+		    	if (writer != null)
+		    		writer.close();
+		    } catch (IOException e) {
+		    	throw new SAXException("Unable to close temp file containing serialized data",e);
+		    }
 		}
     }