Bitcoin Forum
October 24, 2018, 02:18:57 AM *
News: Make sure you are not using versions of Bitcoin Core other than 0.17.0 [Torrent], 0.16.3, 0.15.2, or 0.14.3. More info.
 
   Home   Help Search Donate Login Register  
Pages: [1]
  Print  
Author Topic: Where does the 40Byte OP_RETURN Restriction Exist in the Code?  (Read 1828 times)
SilentCicer0
Newbie
*
Offline Offline

Activity: 6
Merit: 0


View Profile
November 08, 2014, 05:48:11 PM
 #1

Where in the Bitcoin wallet code is the 40byte restriction for OP_RETURN transactions enforced?

So far, I've found material in the code on lines:
328 **src/script/script.h
, 678 **src/main.cpp
, 57 **src/script/standard.cpp

Is the 40byte restriction enforced in the code, or is there something I'm missing here?


... Answered my own question again... I think I found it here:
https://github.com/bitcoin/bitcoin/pull/3737/files


For anyone else who wants to read about OP_RETURN, this post was pretty sweet:
http://bitcoin.stackexchange.com/questions/29554/explanation-of-what-an-op-return-transaction-looks-like


Cheers,
SC.


References:
https://github.com/bitcoin/bitcoin/search?utf8=%E2%9C%93&q=OP_RETURN
https://en.bitcoin.it/wiki/Script
http://coinsecrets.org/
http://bitcoin.stackexchange.com/questions/23024/op-return-data-transaction
1540347537
Hero Member
*
Offline Offline

Posts: 1540347537

View Profile Personal Message (Offline)

Ignore
1540347537
Reply with quote  #2

1540347537
Report to moderator
Advertised sites are not endorsed by the Bitcoin Forum. They may be unsafe, untrustworthy, or illegal in your jurisdiction. Advertise here.
1540347537
Hero Member
*
Offline Offline

Posts: 1540347537

View Profile Personal Message (Offline)

Ignore
1540347537
Reply with quote  #2

1540347537
Report to moderator
1540347537
Hero Member
*
Offline Offline

Posts: 1540347537

View Profile Personal Message (Offline)

Ignore
1540347537
Reply with quote  #2

1540347537
Report to moderator
instagibbs
Member
**
Offline Offline

Activity: 114
Merit: 10


View Profile
November 08, 2014, 09:02:23 PM
 #2

https://github.com/bitcoin/bitcoin/pull/3737

Should be everything needed.
Remember remember the 5th of November
Legendary
*
Offline Offline

Activity: 1778
Merit: 1001

Reverse engineer from time to time


View Profile
November 09, 2014, 02:32:15 PM
 #3

If you actually look a bit closer, 6-7 guys are responsible for whatever gets in the client.

BTC:1AiCRMxgf1ptVQwx6hDuKMu4f7F27QmJC2
altcoinex
Sr. Member
****
Offline Offline

Activity: 293
Merit: 250


Director - www.cubeform.io


View Profile WWW
December 06, 2014, 04:47:56 PM
 #4

https://github.com/bitcoin/bitcoin/pull/3737/files#diff-4f6d60b6976522cec2974a0aea9a5ab3R296 In specific should show the code in question...


                                     ╓╢╬╣╣╖
                                   ┌║██████║∩
                                   ]█████████
                                    ╜██████╝`
                                      ╙╜╜╜`
                                   ╓╥@@@@@@╥╓
         ╓╖@@╖,                 ,@║██████████╢@,                 ,╓@@╖╓
       ╓╢██████╢.              ╓╢███████████████╖               ║╢█████║╓
       ║█████████    ,,╓╓,,   ┌║█████████████████┐   ,,╓╓,,    ]█████████
       └╢██████║` ╓╢║██████╢║∩``╙╙╙╙╙╙╙╙╙╙╙╙╙╙╙╙╙`»╢╢██████╢║╖  ║███████╜
         "╜╜╜╜` ╖╢█████████╣╜                      └╢██████████@ `╜╜╜╜╜
               ║██████████╜                          ╙╢██████████
              ┌█████████╜                              ╙╢█████████
              └███████╨`                                 ╜████████
               ║████╨╜                                    `╢█████
                ╙╢╣╜                                        └╢█╜
                ,,                                            ,,
             ╓@║██┐                                          ┌██║@╓
            ╢██████                                          ]█████H
           ╢███████∩                                        ┌████████
  ╓@@@@╓   █████████                                        ║████████`  ╓@@@@╖
╓╢██████║. █████████∩                                      ┌█████████ ,║███████╖
██████████ └█████████                                      ██████████ ]█████████
`║██████╜`  └╢████████                                    ┌███████╣╜   ╙██████╨`
  `╙╜╜╙`      `╙╨╢████                                    █████╝╜`       `╙╜╜`
                      ]@╓                              ╓╖H
                      ███╢║@╓,                    ,╓@╢╢███`
                      ████████╢@╖╓.           ╓╖@║████████`
                      ]███████████╢║@╓,  ,╓@╢╢████████████
                       ╙╢█████████████╨` ╜██████████████╜
                         ╙╝╢███████║╜`    `╜║████████╝╜`
                     ,╓@@@╓  `²╙``             `╙²`  ╓@@@╖,
                    ║╢█████╢H                      ╓╢██████H
                    █████████                      █████████`
                    ╙╢██████╜                      ╙╢██████╜
                      └╨╩╝┘                          └╨╩╝╜
WINFLOW.
██
██
██
██
██
██
██
██
██
██
██
██
██
..
██
██
██
██
██
██
██
██
██
██
██
██
██
.
dexX7
Legendary
*
Offline Offline

Activity: 1106
Merit: 1000



View Profile WWW
December 08, 2014, 04:04:25 PM
 #5

If you actually look a bit closer, 6-7 guys are responsible for whatever gets in the client.

Likewise, there are a few open PRs addressing this topic:

Relay up to two (2) OP_RETURN outputs as IsStandard
https://github.com/bitcoin/bitcoin/pull/5075

Accept any sequence of PUSHDATAs in OP_RETURN outputs
https://github.com/bitcoin/bitcoin/pull/5079

Change the default maximum OP_RETURN size to 80 bytes
https://github.com/bitcoin/bitcoin/pull/5286

Pages: [1]
  Print  
 
Jump to:  

Sponsored by , a Bitcoin-accepting VPN.
Powered by MySQL Powered by PHP Powered by SMF 1.1.19 | SMF © 2006-2009, Simple Machines Valid XHTML 1.0! Valid CSS!