On Sat, 8 Jul 2023 02:05:27 GMT, Jenny Shivayogi wrote:
> Free-ing 'buf' before two conditional return statements introduced by
> JDK-8307990
This pull request has now been integrated.
Changeset: 401c3dea
Author: Jenny Shivayogi
Committer: Aleksey Shipilev
U
On Mon, 10 Jul 2023 16:38:26 GMT, Volker Simonis wrote:
>> Free-ing 'buf' before two conditional return statements introduced by
>> JDK-8307990
>
> Hi @kspeeyu ,
>
> Thanks for fixing this issue. The fix looks good except the whitespace issue
> already mentioned before.
>
> Notice that we don
On Mon, 10 Jul 2023 11:12:36 GMT, Aleksey Shipilev wrote:
>> Free-ing 'buf' before two conditional return statements introduced by
>> JDK-8307990
>
> src/java.base/unix/native/libjava/ProcessImpl_md.c line 575:
>
>> 573: writeFully(c->childenv[1], buf, bufsize) != bufsize) {
>> 574:
Free-ing 'buf' before two conditional return statements introduced by
JDK-8307990
-
Commit messages:
- removing tab space and resolving merge conflict
- Merge branch 'openjdk:master' into JDK-8311645
- removing tab space
- freeing the buf before return
- removing tab space
Chan
