search

zhangpeihao / gortmp

Implement RTMP protocol by golang
MIT License
523 stars 162 forks source link

This field MUST be all 0s #3

Closed oikomi closed 10 years ago

oikomi commented 10 years ago

hi @zhangpeihao

http://wwwimages.adobe.com/content/dam/Adobe/en/devnet/rtmp/pdf/rtmp_specification_1.0.pdf says that : Zero (4 bytes): This field MUST be all 0s.

look this code

    // Set FlashPlayer version
    for i := 0; i < 4; i++ {
        s1[4+i] = FMS_VERSION[i]
    }

why you set them not zero?

and I also test ok if I set s1[4+i] = 0

zhangpeihao commented 10 years ago

RTMP spec document is out of date. You can find the detail information in this web page: http://en.wikipedia.org/wiki/Real_Time_Messaging_Protocol#Specification_document