diff --git a/ofproto/ofproto-dpif-upcall.h b/ofproto/ofproto-dpif-upcall.h
index efa3cd190..cef1d3419 100644
--- a/ofproto/ofproto-dpif-upcall.h
+++ b/ofproto/ofproto-dpif-upcall.h
@@ -24,7 +24,7 @@ struct ofpbuf;
 struct seq;
 struct simap;
 
-/* Udif is responsible for retrieving upcalls from the kernel and processing
+/* Udpif is responsible for retrieving upcalls from the kernel and processing
  * them.  Additionally, it's responsible for maintaining the datapath flow
  * table. */
 
diff --git a/ofproto/ofproto-dpif-xlate.c b/ofproto/ofproto-dpif-xlate.c
index 9b3a2f28a..a2b4fdb3b 100644
--- a/ofproto/ofproto-dpif-xlate.c
+++ b/ofproto/ofproto-dpif-xlate.c
@@ -1387,7 +1387,7 @@ xlate_lookup_ofproto(const struct dpif_backer *backer, const struct flow *flow,
 }
 
 /* Given a datapath and flow metadata ('backer', and 'flow' respectively),
- * optionally populates 'ofproto' with the ofproto_dpif, 'ofp_in_port' with the
+ * optionally populates 'ofprotop' with the ofproto_dpif, 'ofp_in_port' with the
  * openflow in_port, and 'ipfix', 'sflow', and 'netflow' with the appropriate
  * handles for those protocols if they're enabled.  Caller may use the returned
  * pointers until quiescing, for longer term use additional references must