Fix for problems signaled by clang static analyzer

Former-commit-id: 2ad296acdc783054198a7b2f2f34c36d9cc22958

src/Nazara/Core/Posix/FileImpl.cpp

@@ -143,7 +143,18 @@ namespace Nz
 			return false;
 		}
 
-		mode_t permissions; // TODO : get permission from first file
+		mode_t permissions;
+		struct stat sb;
+		if (fstat(fd1, &sb) == -1) // get permission from first file
+		{
+			NazaraWarning("Could not get permissions of source file");
+			permissions = S_IRUSR | S_IWUSR | S_IRGRP | S_IROTH;
+		}
+		else
+		{
+			permissions = sb.st_mode & ~S_IFMT; // S_IFMT: bit mask for the file type bit field -> ~S_IFMT: general permissions
+		}
+
 		int fd2 = open64(targetPath.GetConstBuffer(), O_WRONLY | O_TRUNC, permissions);
 		if (fd2 == -1)
 		{

src/Nazara/Core/String.cpp

@@ -2827,21 +2827,15 @@ namespace Nz
 			{
 				const char* c = oldCharacters;
 				char character = Detail::ToLower(*ptr);
-				bool found = false;
+
+				std::ptrdiff_t offset = ptr - m_sharedString->string.get();
+				EnsureOwnership();
+				ptr = &m_sharedString->string[offset];
+
 				do
 				{
 					if (character == Detail::ToLower(*c))
 					{
-						if (!found)
-						{
-							std::ptrdiff_t offset = ptr - m_sharedString->string.get();
-
-							EnsureOwnership();
-
-							ptr = &m_sharedString->string[offset];
-							found = true;
-						}
-
 						*ptr = replaceCharacter;
 						++count;
 						break;
@@ -2853,19 +2847,12 @@ namespace Nz
 		}
 		else
 		{
-			bool found = false;
+			std::ptrdiff_t offset = ptr - m_sharedString->string.get();
+			EnsureOwnership();
+			ptr = &m_sharedString->string[offset];
+
 			while ((ptr = std::strpbrk(ptr, oldCharacters)) != nullptr)
 			{
-				if (!found)
-				{
-					std::ptrdiff_t offset = ptr - m_sharedString->string.get();
-
-					EnsureOwnership();
-
-					ptr = &m_sharedString->string[offset];
-					found = true;
-				}
-
 				*ptr++ = replaceCharacter;
 				++count;
 			}

src/Nazara/Graphics/DepthRenderTechnique.cpp

@@ -464,7 +464,6 @@ namespace Nz
 	void DepthRenderTechnique::DrawOpaqueModels(const SceneData& sceneData, ForwardRenderQueue::Layer& layer) const
 	{
 		const Shader* lastShader = nullptr;
-		const ShaderUniforms* shaderUniforms = nullptr;
 
 		for (auto& matIt : layer.opaqueModels)
 		{
@@ -488,7 +487,7 @@ namespace Nz
 					if (shader != lastShader)
 					{
 						// Index of uniforms in the shader
-						shaderUniforms = GetShaderUniforms(shader);
+						GetShaderUniforms(shader);
 						lastShader = shader;
 					}