I am newer to writing triggers and I am not sure if the code below is causing me to receive the infamous Too Many DML Statements 151. Is it not bulkified correctly?

I took this code from what I saw online and just applied it to my situation. Essentially I needed to take the primary contact from my opportunity contact role and place it in a look field on the opportunity page so that I can look at it for workflows and such. Thanks in advance.
 

trigger InsertPrimaryContact on Opportunity (before insert, before update) {

Set<Id> OppIds = new Set<Id>();
for (Opportunity o : Trigger.new) {
    OppIds.add(o.id);}

Map<Id, List<OpportunityContactRole>> Opp_OCR = new Map<Id, List<OpportunityContactRole>>();

for (OpportunityContactRole oppocr : [select id, ContactId, Opportunityid, isPrimary from 
    OpportunityContactRole where opportunityid in :OppIds and isPrimary = true]) {

    List<OpportunityContactRole> tmp_oppocr = new List<OpportunityContactRole>();

    tmp_oppocr = Opp_OCR.get(oppocr.opportunityid);

    if (tmp_oppocr == null) {
        Opp_OCR.put(oppocr.opportunityid, new List<OpportunityContactRole>{oppocr});
    } 
    
    else {

        tmp_oppocr.add(oppocr);
        Opp_OCR.put(oppocr.opportunityid, tmp_oppocr);
    }
} 




system.debug('Final Opp_OCR map: '+Opp_OCR);


for (Opportunity opps : Trigger.new) {
    List<OpportunityContactRole> this_oppocr = new List<OpportunityContactRole>();
    this_oppocr = Opp_OCR.get(opps.id);
    system.debug('this Opps ('+opps.id+') list of oppocrs: '+this_oppocr);

    if (this_oppocr == null) {
        opps.Primary_Contact__c = null;
    }    
    else {

        for (OpportunityContactRole r : this_oppocr) {
            if (r.isprimary) 
                opps.Primary_Contact__c = r.ContactId;
        } 
    } 
} 

}