diff options
| author | Even Rouault <even.rouault@spatialys.com> | 2020-01-23 18:31:32 +0100 |
|---|---|---|
| committer | Even Rouault <even.rouault@spatialys.com> | 2020-01-23 19:11:05 +0100 |
| commit | 79eed0da28abdc41c996dc55ee231f1fcffe4fd2 (patch) | |
| tree | 97e5a634014d57543cd4642c4407e5847597fccc /src | |
| parent | a217239fa55b5c7002d37a5b17fb41117dfb4b64 (diff) | |
| download | PROJ-79eed0da28abdc41c996dc55ee231f1fcffe4fd2.tar.gz PROJ-79eed0da28abdc41c996dc55ee231f1fcffe4fd2.zip | |
Fix wrong use of derivingConversionRef() that caused GDAL bug
Hopefully final cut at solving the same class of bug that the one
that affected QGIS in December.
This time, this hit GDAL in the situation of
https://lists.osgeo.org/pipermail/gdal-dev/2020-January/051500.html
The bug fix for that particular issue is in
PROJStringParser::createFromPROJString()
But grepping more in the code base, I could find other potential smelly
situations (might not be issues, but better be safe than sorry),
so let's fix them too.
Bottom line is:
derivingConversionRef() should *only* be used for consultation, and
never to create a new ProjectedCRS()
Diffstat (limited to 'src')
| -rw-r--r-- | src/iso19111/c_api.cpp | 2 | ||||
| -rw-r--r-- | src/iso19111/factory.cpp | 9 | ||||
| -rw-r--r-- | src/iso19111/io.cpp | 2 |
3 files changed, 6 insertions, 7 deletions
diff --git a/src/iso19111/c_api.cpp b/src/iso19111/c_api.cpp index 5a5e97f6..6fef5518 100644 --- a/src/iso19111/c_api.cpp +++ b/src/iso19111/c_api.cpp @@ -3563,7 +3563,7 @@ PJ *proj_crs_create_projected_3D_crs_from_2D(PJ_CONTEXT *ctx, crs_name ? crs_name : cpp_projected_2D_crs->nameStr().c_str()), NN_NO_CHECK(cpp_geog_3D_CRS), - cpp_projected_2D_crs->derivingConversionRef(), newCS)); + cpp_projected_2D_crs->derivingConversion(), newCS)); } catch (const std::exception &e) { proj_log_error(ctx, __FUNCTION__, e.what()); if (ctx->cpp_context) { diff --git a/src/iso19111/factory.cpp b/src/iso19111/factory.cpp index 4989e634..9ed9dc25 100644 --- a/src/iso19111/factory.cpp +++ b/src/iso19111/factory.cpp @@ -2581,7 +2581,7 @@ AuthorityFactory::createProjectedCRS(const std::string &code) const { pj_add_type_crs_if_needed(text_definition), d->context()); auto projCRS = dynamic_cast<const crs::ProjectedCRS *>(obj.get()); if (projCRS) { - const auto &conv = projCRS->derivingConversionRef(); + const auto conv = projCRS->derivingConversion(); auto newConv = (conv->nameStr() == "unnamed") ? operation::Conversion::create( @@ -2602,10 +2602,9 @@ AuthorityFactory::createProjectedCRS(const std::string &code) const { boundCRS->baseCRS().get()); if (projCRS) { auto newBoundCRS = crs::BoundCRS::create( - crs::ProjectedCRS::create( - props, projCRS->baseCRS(), - projCRS->derivingConversionRef(), - projCRS->coordinateSystem()), + crs::ProjectedCRS::create(props, projCRS->baseCRS(), + projCRS->derivingConversion(), + projCRS->coordinateSystem()), boundCRS->hubCRS(), boundCRS->transformation()); return NN_NO_CHECK( util::nn_dynamic_pointer_cast<crs::ProjectedCRS>( diff --git a/src/iso19111/io.cpp b/src/iso19111/io.cpp index a67238d9..7a3cba6b 100644 --- a/src/iso19111/io.cpp +++ b/src/iso19111/io.cpp @@ -9057,7 +9057,7 @@ PROJStringParser::createFromPROJString(const std::string &projString) { auto projCRS = dynamic_cast<ProjectedCRS *>(crs); if (projCRS) { // Override with easting northing order - const auto &conv = projCRS->derivingConversionRef(); + const auto conv = projCRS->derivingConversion(); if (conv->method()->getEPSGCode() != EPSG_CODE_METHOD_TRANSVERSE_MERCATOR_SOUTH_ORIENTATED) { return ProjectedCRS::create( |