From 3ae9c77e8394eb96d4205b7fa2b3634987b9664f Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=C3=81kos=20Sz=C5=91ts?= <szotsaki@gmail.com>
Date: Wed, 12 Jun 2019 12:23:46 +0200
Subject: [PATCH] Open files in read-only mode in validator

---
 app/json-schema-validate.cpp | 15 +++++++--------
 1 file changed, 7 insertions(+), 8 deletions(-)

diff --git a/app/json-schema-validate.cpp b/app/json-schema-validate.cpp
index e0061d2..b65b7eb 100644
--- a/app/json-schema-validate.cpp
+++ b/app/json-schema-validate.cpp
@@ -24,13 +24,12 @@ static void usage(const char *name)
 static void loader(const json_uri &uri, json &schema)
 {
 	std::string filename = "./" + uri.path();
-	std::fstream lf(filename);
+	std::ifstream lf(filename);
 	if (!lf.good())
 		throw std::invalid_argument("could not open " + uri.url() + " tried with " + filename);
-
 	try {
 		lf >> schema;
-	} catch (std::exception &e) {
+	} catch (const std::exception &e) {
 		throw e;
 	}
 }
@@ -49,7 +48,7 @@ int main(int argc, char *argv[])
 	if (argc != 2)
 		usage(argv[0]);
 
-	std::fstream f(argv[1]);
+	std::ifstream f(argv[1]);
 	if (!f.good()) {
 		std::cerr << "could not open " << argv[1] << " for reading\n";
 		usage(argv[0]);
@@ -59,8 +58,8 @@ int main(int argc, char *argv[])
 	json schema;
 	try {
 		f >> schema;
-	} catch (std::exception &e) {
-		std::cerr << e.what() << " at " << f.tellp() << " - while parsing the schema\n";
+	} catch (const std::exception &e) {
+		std::cerr << e.what() << " at " << f.tellg() << " - while parsing the schema\n";
 		return EXIT_FAILURE;
 	}
 
@@ -71,7 +70,7 @@ int main(int argc, char *argv[])
 		// insert this schema as the root to the validator
 		// this resolves remote-schemas, sub-schemas and references via the given loader-function
 		validator.set_root_schema(schema);
-	} catch (std::exception &e) {
+	} catch (const std::exception &e) {
 		std::cerr << "setting root schema failed\n";
 		std::cerr << e.what() << "\n";
 	}
@@ -81,7 +80,7 @@ int main(int argc, char *argv[])
 
 	try {
 		std::cin >> document;
-	} catch (std::exception &e) {
+	} catch (const std::exception &e) {
 		std::cerr << "json parsing failed: " << e.what() << " at offset: " << std::cin.tellg() << "\n";
 		return EXIT_FAILURE;
 	}