[GRASS-git] [OSGeo/grass] 236559: lib/imagery: Fix Resource Leak Issue in iscatt_cor...

OSGeo Weblate noreply at github.com
Sat Aug 16 08:40:26 PDT 2025


  Branch: refs/heads/periodic/update-configure
  Home:   https://github.com/OSGeo/grass
  Commit: 2365597e801afe540069a70b05367e47c1ffa320
      https://github.com/OSGeo/grass/commit/2365597e801afe540069a70b05367e47c1ffa320
  Author: ShubhamDesai <42180509+ShubhamDesai at users.noreply.github.com>
  Date:   2025-08-09 (Sat, 09 Aug 2025)

  Changed paths:
    M lib/imagery/iscatt_core.c

  Log Message:
  -----------
  lib/imagery: Fix Resource Leak Issue in iscatt_core.c (#6102)


  Commit: 1ccfa1e6cb078a19a188bef52cc155fc3d47aa06
      https://github.com/OSGeo/grass/commit/1ccfa1e6cb078a19a188bef52cc155fc3d47aa06
  Author: ShubhamDesai <42180509+ShubhamDesai at users.noreply.github.com>
  Date:   2025-08-09 (Sat, 09 Aug 2025)

  Changed paths:
    M display/d.vect/legend.c
    M display/d.vect/main.c

  Log Message:
  -----------
  d.vect: Fix Copy into fixed buffer size issues (#6138)


  Commit: bfaad3f4d697a97b0690b3ae2572ecc5f266daa0
      https://github.com/OSGeo/grass/commit/bfaad3f4d697a97b0690b3ae2572ecc5f266daa0
  Author: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
  Date:   2025-08-09 (Sat, 09 Aug 2025)

  Changed paths:
    M locale/templates/grasslibs.pot
    M locale/templates/grassmods.pot
    M locale/templates/grasswxpy.pot

  Log Message:
  -----------
  locale: Update translation files (#6190)

Co-authored-by: echoix <27212526+echoix at users.noreply.github.com>


  Commit: 620c822751d56822a201caa2a7587cff3e5afe0c
      https://github.com/OSGeo/grass/commit/620c822751d56822a201caa2a7587cff3e5afe0c
  Author: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
  Date:   2025-08-09 (Sat, 09 Aug 2025)

  Changed paths:
    M .pre-commit-config.yaml

  Log Message:
  -----------
  CI(deps): Update pre-commit hook pre-commit/pre-commit-hooks to v6 (#6191)


  Commit: 71d952fbddcf06ad27cb9e927bdd1260e88d1304
      https://github.com/OSGeo/grass/commit/71d952fbddcf06ad27cb9e927bdd1260e88d1304
  Author: Mohan Yelugoti <ymdatta.work at gmail.com>
  Date:   2025-08-09 (Sat, 09 Aug 2025)

  Changed paths:
    M imagery/i.smap/model.c

  Log Message:
  -----------
  i.smap: fix possible pole error with log in extract function (#4499)

Using logarithm function call with zero argument will lead to
a pole error, which occurs if the mathematical function has
an exact infinite result.

Refactor the conditional to only execute the code when number
of subclasses are more than 1, which would preemptively stop
us from getting into situation where log would have a zero
argument. In cases where number of subclasses are <= 0, set
the default value to 0. This is necessary as `ll` points to
malloc'd memory and can contain random value.

Signed-off-by: Mohan Yelugoti <ymdatta.work at gmail.com>


  Commit: b572235296b60cd6d373ef3cdcc96bbf9dc6f008
      https://github.com/OSGeo/grass/commit/b572235296b60cd6d373ef3cdcc96bbf9dc6f008
  Author: Edouard Choinière <27212526+echoix at users.noreply.github.com>
  Date:   2025-08-10 (Sun, 10 Aug 2025)

  Changed paths:
    M locale/grass_po_stats.py

  Log Message:
  -----------
  locale: Use context manager for opening files in locale/grass_po_stats (#6192)


  Commit: ce482969ae94a12df16ded33d03cbeddcbab957c
      https://github.com/OSGeo/grass/commit/ce482969ae94a12df16ded33d03cbeddcbab957c
  Author: Markus Neteler <neteler at osgeo.org>
  Date:   2025-08-11 (Mon, 11 Aug 2025)

  Changed paths:
    M gui/wxpython/startup/locdownload.py

  Log Message:
  -----------
  wxGUI: fix URL of gismentors.zip data file (#6193)

Update outdated URL to gismentors.zip.

Source: https://www.training.gismentors.cz/geodata/grass/


  Commit: 9151308c3fb2394c06db508b8423b384dfb4f486
      https://github.com/OSGeo/grass/commit/9151308c3fb2394c06db508b8423b384dfb4f486
  Author: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
  Date:   2025-08-11 (Mon, 11 Aug 2025)

  Changed paths:
    M flake.lock

  Log Message:
  -----------
  CI(deps): Lock file maintenance (#6201)


  Commit: db8d2a9e60a5e3b0056e30fa140a84443d14ccea
      https://github.com/OSGeo/grass/commit/db8d2a9e60a5e3b0056e30fa140a84443d14ccea
  Author: ShubhamDesai <42180509+ShubhamDesai at users.noreply.github.com>
  Date:   2025-08-11 (Mon, 11 Aug 2025)

  Changed paths:
    M lib/vector/Vlib/dangles.c

  Log Message:
  -----------
  lib/vector/Vlib: Fix Resource Leak Issue in dangles.c (#6173)


  Commit: 6df881dd44f1d1807aa812e9a215a779b544c0a7
      https://github.com/OSGeo/grass/commit/6df881dd44f1d1807aa812e9a215a779b544c0a7
  Author: ShubhamDesai <42180509+ShubhamDesai at users.noreply.github.com>
  Date:   2025-08-11 (Mon, 11 Aug 2025)

  Changed paths:
    M lib/vector/Vlib/buffer2.c

  Log Message:
  -----------
  lib/vector/Vlib: Fix Uninitialized Variable issue in buffer2.c (#6174)


  Commit: d0e147205b3340c72b3bdd5c7d5801d28fec8c8c
      https://github.com/OSGeo/grass/commit/d0e147205b3340c72b3bdd5c7d5801d28fec8c8c
  Author: ShubhamDesai <42180509+ShubhamDesai at users.noreply.github.com>
  Date:   2025-08-11 (Mon, 11 Aug 2025)

  Changed paths:
    M raster/r.in.gridatb/file_io.c

  Log Message:
  -----------
  r.in.gridatb: Fix unchecked return value from library in file_io.c (#6149)


  Commit: 43db0c86691ba967874ee422d50fe62a89dccb29
      https://github.com/OSGeo/grass/commit/43db0c86691ba967874ee422d50fe62a89dccb29
  Author: ShubhamDesai <42180509+ShubhamDesai at users.noreply.github.com>
  Date:   2025-08-11 (Mon, 11 Aug 2025)

  Changed paths:
    M vector/v.generalize/misc.c

  Log Message:
  -----------
  v.generalize: Fix Resource Leak Issue in misc.c (#6176)


  Commit: 1031cc9f7fda9e0baef3e38ba2931a3afb17ef40
      https://github.com/OSGeo/grass/commit/1031cc9f7fda9e0baef3e38ba2931a3afb17ef40
  Author: ShubhamDesai <42180509+ShubhamDesai at users.noreply.github.com>
  Date:   2025-08-11 (Mon, 11 Aug 2025)

  Changed paths:
    M display/d.legend.vect/draw.c

  Log Message:
  -----------
  d.legend.vect: Fix Uninitialized Variable Issue in draw.c (#6177)


  Commit: edc87a4920570efe1d80b06033a435fc2f419d8f
      https://github.com/OSGeo/grass/commit/edc87a4920570efe1d80b06033a435fc2f419d8f
  Author: Edouard Choinière <27212526+echoix at users.noreply.github.com>
  Date:   2025-08-11 (Mon, 11 Aug 2025)

  Changed paths:
    M scripts/v.db.addtable/v.db.addtable.py

  Log Message:
  -----------
  v.db.addtable: Use context manager for devnull to solve ResourceWarning (#6195)

The opened file object isn't one that shouldn't be closed, so a simple transformation to use a context manager is appropriate.


  Commit: 2b2096424495456fa71154b8771338b10a6e0c75
      https://github.com/OSGeo/grass/commit/2b2096424495456fa71154b8771338b10a6e0c75
  Author: Edouard Choinière <27212526+echoix at users.noreply.github.com>
  Date:   2025-08-12 (Tue, 12 Aug 2025)

  Changed paths:
    M python/grass/script/core.py

  Log Message:
  -----------
  grass.gunittest: Solve ResourceWarning in gs.parser() coming from sys.stdout (#6196)

The note at the end of [`sys.stdout`'s docs](https://docs.python.org/3/library/sys.html#sys.stdout) mentions that writing binary data to the standard streams should use the underlying binary buffer object. However, that file might be replaced by a file-like object that doesn't support the buffer attribute.

So, inspired by [the code example for `sys.displayhook`](https://docs.python.org/3.13/library/sys.html#sys.displayhook), use the binary buffer object if available or simply write the encoded text otherwise.

This solves multiple ResourceWarnings found in tests that could be traced back to this when enabling python tracemalloc as written in the warning messages. One of the ways is to set the env var `PYTHONTRACEMALLOC` to the number of frames to keep. So, `export PYTHONTRACEMALLOC=15` is enough.


  Commit: 52a061bce67a4c8d0c8079e34296322b99f7a0c5
      https://github.com/OSGeo/grass/commit/52a061bce67a4c8d0c8079e34296322b99f7a0c5
  Author: Edouard Choinière <27212526+echoix at users.noreply.github.com>
  Date:   2025-08-13 (Wed, 13 Aug 2025)

  Changed paths:
    M python/grass/script/core.py

  Log Message:
  -----------
  grass.script.core: Use context manager for Popen call in call() function (#6199)


  Commit: b95de9e6bb85a6c9a515f09e191fd9eb74467dd4
      https://github.com/OSGeo/grass/commit/b95de9e6bb85a6c9a515f09e191fd9eb74467dd4
  Author: Nishant Bansal <nishant.bansal.282003 at gmail.com>
  Date:   2025-08-14 (Thu, 14 Aug 2025)

  Changed paths:
    M vector/v.class/main.c
    M vector/v.class/tests/test_v_class.py
    M vector/v.class/v.class.md

  Log Message:
  -----------
  v.class: Add more test cases and fix out-of-bounds bug (#6205)

v.class: add more test cases and fix out-of-bounds bug

Signed-off-by: Nishant Bansal <nishant.bansal.282003 at gmail.com>


  Commit: b6c0c318984cfd4c0b577551fe81cab0311da467
      https://github.com/OSGeo/grass/commit/b6c0c318984cfd4c0b577551fe81cab0311da467
  Author: Vaclav Petras <wenzeslaus at gmail.com>
  Date:   2025-08-14 (Thu, 14 Aug 2025)

  Changed paths:
    M scripts/v.dissolve/v.dissolve.py

  Log Message:
  -----------
  grass.tools: Use Tools in v.dissolve (#6148)

This uses grass.tools in the v.dissolve tool for whatever was previously done by the grass.script.run_command functions. There is no strong reason to do this migration except to test grass.tool more and create more examples, but that's a good reason. Nice thing is that simple calls once the Tools object is created are shorter by one line. Tools objects need to be created. (I opted for creating them again in functions as opposed to passing one to all functions.)

To keep the behavior close to the original, some of the Tools objects do not capture stdout and stderr so that it is printed as it would be with run_command. This means that progress messages will be printed from the underlying tools and error messages will be part of that output as well. (This choice might have been different for a newly written tool or for different parts of a tool.) Places which capture stdin for further processing are kept with the defaults, so stderr is captured and will be part of an exception, but will not be visible otherwise (but was previously visible).

Places which use tool wrappers from grass.script are left as is. Replacement of the convenience wrappers is waiting for JSON outputs of specific tools, so the current change is only focusing on run_command and friends.

Running the pytest test takes the same time with the old and new code (20-29s depending on external factors but consistent for both versions).


  Commit: e70e0919bd5b163470f05a18be4dd602d1a72561
      https://github.com/OSGeo/grass/commit/e70e0919bd5b163470f05a18be4dd602d1a72561
  Author: Edouard Choinière <27212526+echoix at users.noreply.github.com>
  Date:   2025-08-15 (Fri, 15 Aug 2025)

  Changed paths:
    M python/grass/gunittest/gmodules.py
    M python/grass/gunittest/reporters.py
    M python/grass/script/core.py

  Log Message:
  -----------
  grass.gunittest: Use context manager for Popen calls in gunittest (#6198)


  Commit: 76179c43ec87a1a85fd5aadf5d53a45926ad3067
      https://github.com/OSGeo/grass/commit/76179c43ec87a1a85fd5aadf5d53a45926ad3067
  Author: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
  Date:   2025-08-15 (Fri, 15 Aug 2025)

  Changed paths:
    M .github/workflows/additional_checks.yml
    M .github/workflows/clang-format-check.yml
    M .github/workflows/cmake.yml
    M .github/workflows/codeql-analysis.yml
    M .github/workflows/coverity.yml
    M .github/workflows/create_release_draft.yml
    M .github/workflows/docker.yml
    M .github/workflows/documentation.yml
    M .github/workflows/gcc.yml
    M .github/workflows/macos.yml
    M .github/workflows/macos_distribute_app.yml
    M .github/workflows/osgeo4w.yml
    M .github/workflows/periodic_update.yml
    M .github/workflows/pytest.yml
    M .github/workflows/python-code-quality.yml
    M .github/workflows/super-linter.yml
    M .github/workflows/test-nix.yml
    M .github/workflows/titles.yml
    M .github/workflows/ubuntu.yml

  Log Message:
  -----------
  CI(deps): Update actions/checkout action to v5 (#6211)


  Commit: b453dd9cfbcfe4c4422d4ebdd6ea669a7ba73658
      https://github.com/OSGeo/grass/commit/b453dd9cfbcfe4c4422d4ebdd6ea669a7ba73658
  Author: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
  Date:   2025-08-15 (Fri, 15 Aug 2025)

  Changed paths:
    M .github/workflows/additional_checks.yml

  Log Message:
  -----------
  CI(deps): Update astral-sh/setup-uv action to v6.5.0 (#6213)


  Commit: c359c5d0576b3542799c40951a50af63595d3947
      https://github.com/OSGeo/grass/commit/c359c5d0576b3542799c40951a50af63595d3947
  Author: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
  Date:   2025-08-15 (Fri, 15 Aug 2025)

  Changed paths:
    M .github/workflows/codeql-analysis.yml
    M .github/workflows/python-code-quality.yml

  Log Message:
  -----------
  CI(deps): Update github/codeql-action action to v3.29.9 (#6212)


  Commit: adf5b52387b30b72929c7bbb4110f1fd4960dc91
      https://github.com/OSGeo/grass/commit/adf5b52387b30b72929c7bbb4110f1fd4960dc91
  Author: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
  Date:   2025-08-16 (Sat, 16 Aug 2025)

  Changed paths:
    M .pre-commit-config.yaml

  Log Message:
  -----------
  CI(deps): Update pre-commit hook editorconfig-checker/editorconfig-checker to v3.4.0 (#6215)


  Commit: dff8cb9a66b85c9e3f48728c342f711343547caa
      https://github.com/OSGeo/grass/commit/dff8cb9a66b85c9e3f48728c342f711343547caa
  Author: Edouard Choinière <27212526+echoix at users.noreply.github.com>
  Date:   2025-08-15 (Fri, 15 Aug 2025)

  Changed paths:
    M python/grass/gunittest/invoker.py
    M python/grass/gunittest/multirunner.py

  Log Message:
  -----------
  grass.gunittest: Remove Python2-only `-tt` interpreter option (#6194)


  Commit: 82651aff92bbda1a65277edb75a78aca7a27171c
      https://github.com/OSGeo/grass/commit/82651aff92bbda1a65277edb75a78aca7a27171c
  Author: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
  Date:   2025-08-16 (Sat, 16 Aug 2025)

  Changed paths:
    M .github/workflows/python-code-quality.yml
    M .pre-commit-config.yaml

  Log Message:
  -----------
  CI(deps): Update ruff to v0.12.9 (#6217)


  Commit: df2b800b2091828ada4a8516b1ac94d217908cc6
      https://github.com/OSGeo/grass/commit/df2b800b2091828ada4a8516b1ac94d217908cc6
  Author: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
  Date:   2025-08-16 (Sat, 16 Aug 2025)

  Changed paths:
    M Dockerfile
    M docker/ubuntu/Dockerfile
    M docker/ubuntu_wxgui/Dockerfile

  Log Message:
  -----------
  CI(deps): Update ubuntu:22.04 Docker digest to 1aa979d (#6216)


  Commit: 2c0a3773e11f2326a0f0a3b6afa496df003a07fd
      https://github.com/OSGeo/grass/commit/2c0a3773e11f2326a0f0a3b6afa496df003a07fd
  Author: OSGeo Weblate <97247866+osgeoweblate at users.noreply.github.com>
  Date:   2025-08-16 (Sat, 16 Aug 2025)

  Changed paths:
    M locale/po/grasslibs_ar.po
    M locale/po/grasslibs_bn.po
    M locale/po/grasslibs_cs.po
    M locale/po/grasslibs_de.po
    M locale/po/grasslibs_el.po
    M locale/po/grasslibs_es.po
    M locale/po/grasslibs_fi.po
    M locale/po/grasslibs_fr.po
    M locale/po/grasslibs_hu.po
    M locale/po/grasslibs_id_ID.po
    M locale/po/grasslibs_it.po
    M locale/po/grasslibs_ja.po
    M locale/po/grasslibs_ko.po
    M locale/po/grasslibs_lv.po
    M locale/po/grasslibs_ml.po
    M locale/po/grasslibs_pl.po
    M locale/po/grasslibs_pt.po
    M locale/po/grasslibs_pt_BR.po
    M locale/po/grasslibs_ro.po
    M locale/po/grasslibs_ru.po
    M locale/po/grasslibs_si.po
    M locale/po/grasslibs_sl.po
    M locale/po/grasslibs_sv.po
    M locale/po/grasslibs_ta.po
    M locale/po/grasslibs_th.po
    M locale/po/grasslibs_tr.po
    M locale/po/grasslibs_uk.po
    M locale/po/grasslibs_vi.po
    M locale/po/grasslibs_zh.po
    M locale/po/grassmods_ar.po
    M locale/po/grassmods_bn.po
    M locale/po/grassmods_cs.po
    M locale/po/grassmods_de.po
    M locale/po/grassmods_el.po
    M locale/po/grassmods_es.po
    M locale/po/grassmods_fi.po
    M locale/po/grassmods_fr.po
    M locale/po/grassmods_hu.po
    M locale/po/grassmods_id_ID.po
    M locale/po/grassmods_it.po
    M locale/po/grassmods_ja.po
    M locale/po/grassmods_ko.po
    M locale/po/grassmods_lv.po
    M locale/po/grassmods_ml.po
    M locale/po/grassmods_pl.po
    M locale/po/grassmods_pt.po
    M locale/po/grassmods_pt_BR.po
    M locale/po/grassmods_ro.po
    M locale/po/grassmods_ru.po
    M locale/po/grassmods_si.po
    M locale/po/grassmods_sl.po
    M locale/po/grassmods_sv.po
    M locale/po/grassmods_ta.po
    M locale/po/grassmods_th.po
    M locale/po/grassmods_tr.po
    M locale/po/grassmods_uk.po
    M locale/po/grassmods_vi.po
    M locale/po/grassmods_zh.po
    M locale/po/grasswxpy_ar.po
    M locale/po/grasswxpy_bn.po
    M locale/po/grasswxpy_cs.po
    M locale/po/grasswxpy_de.po
    M locale/po/grasswxpy_el.po
    M locale/po/grasswxpy_es.po
    M locale/po/grasswxpy_fi.po
    M locale/po/grasswxpy_fr.po
    M locale/po/grasswxpy_hu.po
    M locale/po/grasswxpy_id_ID.po
    M locale/po/grasswxpy_it.po
    M locale/po/grasswxpy_ja.po
    M locale/po/grasswxpy_ko.po
    M locale/po/grasswxpy_lv.po
    M locale/po/grasswxpy_ml.po
    M locale/po/grasswxpy_pl.po
    M locale/po/grasswxpy_pt.po
    M locale/po/grasswxpy_pt_BR.po
    M locale/po/grasswxpy_ro.po
    M locale/po/grasswxpy_ru.po
    M locale/po/grasswxpy_si.po
    M locale/po/grasswxpy_sl.po
    M locale/po/grasswxpy_sv.po
    M locale/po/grasswxpy_ta.po
    M locale/po/grasswxpy_th.po
    M locale/po/grasswxpy_tr.po
    M locale/po/grasswxpy_uk.po
    M locale/po/grasswxpy_vi.po
    M locale/po/grasswxpy_zh.po

  Log Message:
  -----------
  Translations update from OSGeo Weblate (#6219)


Compare: https://github.com/OSGeo/grass/compare/5a84ee6dd209...2c0a3773e11f

To unsubscribe from these emails, change your notification settings at https://github.com/OSGeo/grass/settings/notifications


More information about the grass-commit mailing list