RFR: 8320219: Actually resolve issues with goto labels in sspi [v12]
    Julian Waters 
    jwaters at openjdk.org
       
    Fri Apr  5 06:31:16 UTC 2024
    
    
  
> I regret not actually addressing the issues with the goto labels in https://github.com/openjdk/jdk/pull/15996, where initialization of locals in sspi were jumped over by gotos to a certain label. I changed the initializations into split declarations and assignments in https://github.com/openjdk/jdk/pull/15996, but this is simply a hack and does not address the real issue of gotos jumping over locals. I've as such fixed the issues with them properly this time, by simply deleting the labels and duplicating the code where they're used. As mentioned, this unfortunately does increase duplicate code, but is the cleanest solution I could come up with for the labels
Julian Waters has updated the pull request incrementally with two additional commits since the last revision:
 - Include memory header in sspi.cpp
 - RAII in sspi.cpp
-------------
Changes:
  - all: https://git.openjdk.org/jdk/pull/16682/files
  - new: https://git.openjdk.org/jdk/pull/16682/files/cc9992e5..c419cc29
Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=16682&range=11
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=16682&range=10-11
  Stats: 16 lines in 1 file changed: 1 ins; 3 del; 12 mod
  Patch: https://git.openjdk.org/jdk/pull/16682.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/16682/head:pull/16682
PR: https://git.openjdk.org/jdk/pull/16682
    
    
More information about the security-dev
mailing list