Skip to content

v.out.ogr: Fix OGR_L_GetLayerDefn() result - #8071

Merged
echoix merged 1 commit into
OSGeo:mainfrom
ptitjano:v-out-ogr-feature-defn
Oct 15, 2026
Merged

echoix merged 1 commit into
OSGeo:mainfrom
ptitjano:v-out-ogr-feature-defn

Conversation

@ptitjano

Copy link
Copy Markdown
Contributor

OGR_L_GetLayerDefn() returns an OGRFeatureDefnH, but the result was stored in an OGRFieldDefnH.

This error only occurs when GDAL is built with GDAL_DEBUG.

@github-actions github-actions Bot added vector Related to vector data processing C Related code is in C module labels Oct 14, 2026
@ptitjano

Copy link
Copy Markdown
Contributor Author

I found this issue while working on the vcpkg port of grass. See: microsoft/vcpkg#54374

@echoix

echoix commented Oct 14, 2026

Copy link
Copy Markdown
Member

This will need a bit of rework to not fail existing workflows

@nilason nilason left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just after quick read, the following changes are probably needed.

Comment thread vector/v.out.ogr/main.c Outdated
Comment thread vector/v.out.ogr/main.c Outdated
@ptitjano
ptitjano force-pushed the v-out-ogr-feature-defn branch 3 times, most recently from dd84be6 to 0e48193 Compare October 15, 2026 07:59
Comment thread vector/v.out.ogr/main.c Outdated
`OGR_L_GetLayerDefn()` returns an `OGRFeatureDefnH`, but the result
was stored in an `OGRFieldDefnH`. This error only occurs when GDAL
is built with GDAL_DEBUG.
@ptitjano
ptitjano force-pushed the v-out-ogr-feature-defn branch from 0e48193 to 7612b72 Compare October 15, 2026 08:04
@ptitjano

Copy link
Copy Markdown
Contributor Author

Just after quick read, the following changes are probably needed.

Indeed. This should be fixed now. Thanks.

@echoix
echoix merged commit 5fdda25 into OSGeo:main Oct 15, 2026
26 checks passed
@github-actions github-actions Bot added this to the 8.6.0 milestone Oct 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

C Related code is in C module vector Related to vector data processing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants