RTC: Check error for send_rtcp_fb_pli

trunk/src/app/srs_app_rtc_conn.cpp

@@ -2226,10 +2226,7 @@ srs_error_t SrsRtcConnection::send_rtcp_fb_pli(uint32_t ssrc)
         return srs_error_wrap(err, "protect rtcp psfb pli");
     }
 
-    // TDOO: FIXME: Check error.
+    return sendonly_skt->sendto(protected_buf, nb_protected_buf, 0);
-    sendonly_skt->sendto(protected_buf, nb_protected_buf, 0);
-
-    return err;
 }
 
 void SrsRtcConnection::simulate_nack_drop(int nn)

trunk/src/app/srs_app_rtc_source.cpp

@@ -1704,8 +1704,11 @@ srs_error_t SrsRtcVideoRecvTrack::on_rtp(SrsRtcStream* source, SrsRtpPacket2* pk
     if (request_key_frame_) {
         // TODO: FIXME: add coroutine to request key frame.
         request_key_frame_ = false;
-        // TODO: FIXME: Check error.
+
-        session_->send_rtcp_fb_pli(track_desc_->ssrc_);
+        if ((err = session_->send_rtcp_fb_pli(track_desc_->ssrc_)) != srs_success) {
+            srs_warn("PLI err %s", srs_error_desc(err).c_str());
+            srs_freep(err);
+        }
     }
 
     // For NACK to handle packet.