From e07ab7fb9b5e49093f84b05fb6cecfa00f1b0f52 Mon Sep 17 00:00:00 2001
From: Arno Zeller <azeller@openjdk.org>
Date: Thu, 26 Mar 2026 11:17:18 +0000
Subject: [PATCH] 8380288: [BACKOUT] Incorrect Interpretation of POSIX TZ
 Environment Variable on AIX

Reviewed-by: mbaesken, mdoerr
---
 .../unix/native/libjava/TimeZone_md.c         | 66 +++++++------------
 1 file changed, 22 insertions(+), 44 deletions(-)

diff --git a/src/java.base/unix/native/libjava/TimeZone_md.c b/src/java.base/unix/native/libjava/TimeZone_md.c
index 39e7b726220..cd253edde60 100644
--- a/src/java.base/unix/native/libjava/TimeZone_md.c
+++ b/src/java.base/unix/native/libjava/TimeZone_md.c
@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 1999, 2026, Oracle and/or its affiliates. All rights reserved.
+ * Copyright (c) 1999, 2025, Oracle and/or its affiliates. All rights reserved.
  * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
  *
  * This code is free software; you can redistribute it and/or modify it
@@ -352,15 +352,33 @@ getPlatformTimeZoneID()
 }
 
 static char *
-getJavaTimezoneFromPlatform(const char *tz_buf, size_t tz_len, const char *mapfilename) {
+mapPlatformToJavaTimezone(const char *java_home_dir, const char *tz) {
     FILE *tzmapf;
+    char mapfilename[PATH_MAX + 1];
     char line[256];
     int linecount = 0;
+    char *tz_buf = NULL;
+    char *temp_tz = NULL;
     char *javatz = NULL;
+    size_t tz_len = 0;
 
+    /* On AIX, the TZ environment variable may end with a comma
+     * followed by modifier fields until early AIX6.1.
+     * This restriction has been removed from AIX7. */
+
+    tz_buf = strdup(tz);
+    tz_len = strlen(tz_buf);
+
+    /* Open tzmappings file, with buffer overrun check */
+    if ((strlen(java_home_dir) + 15) > PATH_MAX) {
+        jio_fprintf(stderr, "Path %s/lib/tzmappings exceeds maximum path length\n", java_home_dir);
+        goto tzerr;
+    }
+    strcpy(mapfilename, java_home_dir);
+    strcat(mapfilename, "/lib/tzmappings");
     if ((tzmapf = fopen(mapfilename, "r")) == NULL) {
         jio_fprintf(stderr, "can't open %s\n", mapfilename);
-        return NULL;
+        goto tzerr;
     }
 
     while (fgets(line, sizeof(line), tzmapf) != NULL) {
@@ -413,50 +431,10 @@ getJavaTimezoneFromPlatform(const char *tz_buf, size_t tz_len, const char *mapfi
             break;
         }
     }
-
     (void) fclose(tzmapf);
-    return javatz;
-}
-
-static char *
-mapPlatformToJavaTimezone(const char *java_home_dir, const char *tz) {
-    char mapfilename[PATH_MAX + 1];
-    char *tz_buf = NULL;
-    char *javatz = NULL;
-    char *temp_tz = NULL;
-    size_t tz_len = 0;
-
-    /* On AIX, the TZ environment variable may end with a comma
-     * followed by modifier fields until early AIX6.1.
-     * This restriction has been removed from AIX7. */
-
-    tz_buf = strdup(tz);
-    tz_len = strlen(tz_buf);
-
-    /* Open tzmappings file, with buffer overrun check */
-    if ((strlen(java_home_dir) + 15) > PATH_MAX) {
-        jio_fprintf(stderr, "Path %s/lib/tzmappings exceeds maximum path length\n", java_home_dir);
-        goto tzerr;
-    }
-    strcpy(mapfilename, java_home_dir);
-    strcat(mapfilename, "/lib/tzmappings");
-
-    // First attempt to find the Java timezone for the full tz string
-    javatz = getJavaTimezoneFromPlatform(tz_buf, tz_len, mapfilename);
-
-    // If no match was found, check for timezone with truncated value
-    if (javatz == NULL) {
-        temp_tz = strchr(tz, ',');
-        tz_len = (temp_tz == NULL) ? strlen(tz) : temp_tz - tz;
-        free((void *) tz_buf);
-        tz_buf = (char *)malloc(tz_len + 1);
-        memcpy(tz_buf, tz, tz_len);
-        tz_buf[tz_len] = '\0';
-        javatz = getJavaTimezoneFromPlatform(tz_buf, tz_len, mapfilename);
-    }
 
 tzerr:
-    if (tz_buf != NULL) {
+    if (tz_buf != NULL ) {
         free((void *) tz_buf);
     }