review comments

Author: asoliman92

chains/evm/contracts/interfaces/IRMN.sol

@@ -7,11 +7,11 @@ interface IRMN {
   /// @return subjects the list of cursed subjects.
   function getCursedSubjects() external view returns (bytes16[] memory subjects);
 
-  /// @notice If there is an active global or legacy curse, this function returns true.
+  /// @notice Iff there is an active global or legacy curse, this function returns true.
   /// @return bool true if there is an active global curse.
   function isCursed() external view returns (bool);
 
-  /// @notice If there is an active global curse, or an active curse for `subject`, this function returns true.
+  /// @notice Iff there is an active global curse, or an active curse for `subject`, this function returns true.
   /// @param subject To check whether a particular chain is cursed, set to bytes16(uint128(chainSelector)).
   /// @return bool true if the provided subject is cursed *or* if there is an active global curse.
   function isCursed(

chains/evm/contracts/rmn/RMN.sol

@@ -36,7 +36,7 @@ contract RMN is AuthorizedCallers, ITypeAndVersion, IRMN {
   // ================================================================
 
   /// @notice Curse a single subject.
-  /// @param subject the subject to curse.
+  /// @param subject The subject to curse.
   function curse(
     bytes16 subject
   ) external {
@@ -51,7 +51,11 @@ contract RMN is AuthorizedCallers, ITypeAndVersion, IRMN {
   function curse(
     bytes16[] memory subjects
   ) public {
-    if (msg.sender != owner()) _validateCaller();
+    // Allow both the owner and authorized callers to curse subjects.
+    // Skip validation for the owner; validate authorization for others.
+    if (msg.sender != owner()) {
+        _validateCaller();
+    }
     for (uint256 i = 0; i < subjects.length; ++i) {
       if (!s_cursedSubjects.add(subjects[i])) {
         revert AlreadyCursed(subjects[i]);