com.newrelic.weave.utils.SynchronizedAnnotationNode Maven / Gradle / Ivy

Go to download

Show more of this group Show more artifacts with this name
Show all versions of newrelic-weaver Show documentation

The Weaver of the Java agent.

There is a newer version: 8.17.0

/*
 *
 *  * Copyright 2020 New Relic Corporation. All rights reserved.
 *  * SPDX-License-Identifier: Apache-2.0
 *
 */

package com.newrelic.weave.utils;

import org.objectweb.asm.AnnotationVisitor;
import org.objectweb.asm.tree.AnnotationNode;

/**
 * This class exists only to work around the fact that AnnotationNode instances are NOT thread safe. If multiple threads
 * call {@link #accept(AnnotationVisitor)} at the same time there is a race condition that exists that has been shown to
 * cause VerifyErrors and other serious bytecode-related exceptions.
 * 
 * By synchronizing on the {@link #accept(AnnotationVisitor)} method we are ensuring that any usage of this
 * AnnotationNode in the agent can be accessed by multiple threads. One important thing to note is that this ties us
 * pretty strongly to an ASM version due to the fact that we had to override constructors in the original asm class. If
 * these constructors change or other accept() methods are added we will need to update accordingly.
 * 
 * NOTE: If you are upgrading from org.ow2.asm:asm / org.ow2.asm:asm-tree make sure to double-check that this class hasn't changed between versions.
 */
public class SynchronizedAnnotationNode extends AnnotationNode {

    public SynchronizedAnnotationNode(String desc) {
        super(WeaveUtils.ASM_API_LEVEL, desc);
    }

    public SynchronizedAnnotationNode(int api, String desc) {
        super(api, desc);
    }

    /**
     * This method is synchronized in order to prevent issues with bytecode modification that
     * occur when multiple threads call accept on the same AnnotationNode instance.
     */
    @Override
    public synchronized void accept(AnnotationVisitor av) {
        super.accept(av);
    }

}