diff --git a/awsapi/src/com/cloud/bridge/service/controller/s3/S3BucketAction.java b/awsapi/src/com/cloud/bridge/service/controller/s3/S3BucketAction.java index 6f6f12fae85..a0077a93e61 100644 --- a/awsapi/src/com/cloud/bridge/service/controller/s3/S3BucketAction.java +++ b/awsapi/src/com/cloud/bridge/service/controller/s3/S3BucketAction.java @@ -24,6 +24,7 @@ import java.io.OutputStream; import java.io.Reader; import java.io.StringWriter; import java.io.Writer; +import java.sql.SQLException; import java.text.SimpleDateFormat; import java.util.Calendar; @@ -649,12 +650,12 @@ public class S3BucketAction implements ServletAction { if (null == keyMarker) xml.append(""); else - xml.append("").append(keyMarker).append("").append(StringEscapeUtils.escapeHtml(keyMarker)).append(""); else - xml.append("").append(StringEscapeUtils.escapeHtml(keyMarker)).append("").append(StringEscapeUtils.escapeHtml(versionIdMarker)).append("").append(engineResponse.getMaxKeys()).append(""); xml.append("").append(engineResponse.isTruncated()).append(""); @@ -958,7 +959,7 @@ public class S3BucketAction implements ServletAction { OrderedPair result = uploadDao.getInitiatedUploads(bucketName, maxUploads, prefix, keyMarker, uploadIdMarker); uploads = result.getFirst(); isTruncated = result.getSecond().booleanValue(); - } catch (Exception e) { + } catch (InstantiationException | IllegalAccessException | ClassNotFoundException | SQLException e) { logger.error("List Multipart Uploads failed due to " + e.getMessage(), e); response.setStatus(500); } @@ -966,9 +967,9 @@ public class S3BucketAction implements ServletAction { StringBuffer xml = new StringBuffer(); xml.append(""); xml.append(""); - xml.append("").append(bucketName).append(""); + xml.append("").append(StringEscapeUtils.escapeHtml(bucketName)).append(""); xml.append("").append((null == keyMarker ? "" : StringEscapeUtils.escapeHtml(keyMarker))).append(""); - xml.append("").append((null == uploadIdMarker ? "" : uploadIdMarker)).append(""); + xml.append("").append((null == uploadIdMarker ? "" : StringEscapeUtils.escapeHtml(uploadIdMarker))).append(""); // [C] Construct the contents of the element StringBuffer partsList = new StringBuffer();