Skip to content

src: remove usages of GetBackingStore in crypto

Rodrigo Muino Tomonari requested to merge github/fork/kvakil/crypto-backingstores into main

This removes all usages of GetBackingStore in crypto. See the linked issue for an explanation.

Note: I am not sure of the lifetime semantics intended by ArrayBufferOrViewContents -- I am pretty sure it is correct based on a manual audit of the callsites & by analogy to ArrayBufferViewContents, but please ensure that it is correct. If we are at all unsure of the safety here, I would prefer to abandon this change.

Refs: https://github.com/nodejs/node/issues/32226 Refs: https://github.com/nodejs/node/pull/43921

Merge request reports