Add _ENCODE_FILE_NEW and _ENCODE_FILE_EXISTING variables#105
Conversation
Signed-off-by: sabi789 <sabithac6298@gmail.com>
🤖 Augment PR SummarySummary: This PR extends the z/OS (MVS) Bash port to expose two additional encoding-related special variables. Changes:
Technical Notes: The new hooks aim to influence both child process environments and encoding behavior for file operations on z/OS. 🤖 Was this summary useful? React with 👍 or 👎 |
| + | ||
| + value = value_cell (v); | ||
| + /* Set the environment variable for child processes and file operations */ | ||
| + if (setenv("_ENCODE_FILE_NEW", value, 1) != 0) |
There was a problem hiding this comment.
setenv/unsetenv here may conflict with Bash’s own export_env/environ management (Bash can point environ at an internal buffer), which can lead to stale pointers or child processes not seeing the updated values. Other locations where this applies: stable-patches/variable.patch:68, stable-patches/variable.patch:91, stable-patches/variable.patch:97.
Severity: high
Other Locations
stable-patches/variable.patch:68stable-patches/variable.patch:91stable-patches/variable.patch:97
🤖 Was this useful? React with 👍 or 👎, or 🚀 if it prevented an incident/outage.
|
Do we support _ENCODE_FILE_EXISTING in zoslib now? |
Signed-off-by: sabi789 <sabithac6298@gmail.com>
Signed-off-by: sabi789 <sabithac6298@gmail.com>
|
:/home/sabitha/zostools/zopen_sabi/bashport>env | grep _ENCODE_FILE |
Signed-off-by: sabi789 <sabithac6298@gmail.com>
No description provided.