-
hyc
^ it's patched but we think we can do a better solution given some more time
-
yrt
I am bit confused i used the same function generations commitment for say producing two public key and adding them. That is ge_scalar.....() But when I am using it in my code it is giving zeroes nothing else.
-
yrt
Is anyone here.???
-
yrt
That function is ge_double_scalarmult_base_vartime and ge_tobytes to produce and convert coordinates into key. But the answer it is giving me is zeros
-
rbrunner
Yes, but maybe again nobody competent yet ...
-
yrt
Wht
-
rbrunner
Could you watch the effect in realtime in the debugger? Maybe something is not as you assume, so better to check variables in the debugger as they go in and out of methods?
-
yrt
I did it checking it at every step but it's maths is doing something else that it should not and i am not getting why
-
yrt
But everything is perfect than also it reacting in such a way i tested it for 2 days but now brain is not working and i am not getting why it is giving zeroes
-
rbrunner
Well, as a bug in those methods themselves can be ruled out almost for sure, it almost has to be something on your side - whatever it is.
-
yrt
Yes that's the thing I also thought as i compiled the whole function as it was in source code so it couldn't be wrong but than i checked mine too it also can't be wrong as i only supplied input a and b and nothing else
-
yrt
Sorry not compiled but copied i wanted to test how it worked underneath do i copied the code from sc that did the work and made a program which supplied input which are binding factor and another amount nothing more.
-
rbrunner
Maybe paste your source somewhere, e.g. on
paste.debian.net, so somebody in the know (probably not me) can have a look when they come online later
-
yrt
Ohk i will try it
-
rbrunner
Only the relevant snippets of course :)
-
yrt
But it's monero maths will debian folks understand it.
-
yrt
But wht goes in trying
-
rbrunner
No, paste your source there and then post a link to it here. That's the usual way to show any longer text here, because posting directly here does not turn out well
-
rbrunner
Debian folks just offer the paste service
-
Guest36
How how these debian people will answer me
-
ofrnxmr[m]
Post the link to the paste here
-
moneromooo
You're posting on libera.chat IRC, and it's not the libera people answering. Same deal here.
-
moneromooo
Like when you use your phone to call a friend. It's not the phone company that answers, they're just a conduit.
-
moneromooo
A better comparison is imgur. You put stuff there for others to see, not for the imgur people to see.
-
Guest36
No i wanted to ask from where will I know anyone answered as on debian paste webpage i can't see anything related to that
-
moneromooo
As a generic tip, I suggest using ASAN with your test code. Helps find bugs even when not looking.
-
moneromooo
People who reply will presumably reply in this channel.
-
moneromooo
So... stay around and read.
-
moneromooo
It might also be nobody will know or care to spend time on it if it needs work.
-
moneromooo
(ASAN or valgrind, the latter's speed doesn't matter on trivial test code).
-
rbrunner
Like so: I pasted about 50 lines of this discussion on paste.debian.net, and now I give you the link so you can check:
paste.debian.net/1248710
-
rbrunner
After you pasted the relevant parts of your source code there, and posted a link to it here, people *here* can have a look, and maybe see something
-
rbrunner
And if they do they answer *here* :)
-
Guest36
moneromoo: Are you that same moneromoo that checks and verify vulnerability found by bounty hunters on hackerone.
-
moneromooo
Used to. I barely go there anymore.
-
rbrunner
With 3 o's for added emphasis, not only 2 :)
-
Guest36
rbrunner: Thanks a lot
-
Guest36
moneromoo: okay
-
Guest36
-
Guest36
that's your's
-
rbrunner
Yep.
-
Guest36
-
rbrunner
I guess the relevant part is "main" at the very end, because the other code is just copied over from Monero source, right?
-
Guest36
Yes
-
rbrunner
Alright, I think now everything is ready for people who know such stuff can have a look, and maybe see something about a problem.
-
Guest36
There is nothing wrong than also it output only zeroes
-
Guest36
Okay
-
moneromooo
%p prints a pointer, you probably want %02x or something. Also, you use &sv8.bytes, I *think* it's OK but you mught want to try just sv8.bytes to make sure.
-
yrt
I tried it it not gave any compilation error but their is no difference between wht i wrote now also it's giving zeroes only, do something for those zeroes thanks %02x present the picture in much better way, but wht about zeroes they are here only.
-
moneromooo
Did you try asan or valgrind ?
-
yrt
No
-
moneromooo
Well, I compiled it. You did not use -Wall -W, right ?
-
yrt
So did it gave any answer other than 0
-
moneromooo
Anyway, I ran it, and I do not get only zeroes.
-
yrt
I also compiled with -Wall but got only zeroes
-
moneromooo
I case the values your print to unsigned char too, they're signed otherwise.
-
moneromooo
valgrind moans. Compile with debug info, run with valgrind, fix.
-
yrt
Okay i will try that too did it worked for you
-
moneromooo
Nice. A warning with O2 tells me fe is 10, not 40, long.
-
moneromooo
You probably used 40 due to sizeof, but they're int32_t fields.
-
yrt
Asan is showing many errors of overflow in int32_t but i not used any they were all of monero sc
-
moneromooo
Probably the 10/40 mixup ?
-
yrt
No fe is int32_t fe[] as in monero sc i not touched anything
-
yrt
Fe is array of 10 int32_t
-
yrt
But i not touched any values only used crypto-ops.c and crypto-ops.h from monero sc
-
moneromooo
Well, as I said. I currently modified your code to read like this: for(int i=0;i<10;i++) { printf("X:%02x ",(unsigned char)rv.X[i]);...
-
moneromooo
And valgrind moans no more.
-
Guest36
Wht about the key stored in key hell that's what stores the end input
-
Guest36
*output
-
moneromooo
Not zeroes here either.
-
Guest36
I applied the suggestion but zeroes are here while printing hell.bytes it outputs only zero
-
Guest36
And now everything is zero
-
Guest36
Could you past your code and compiled command and compiled result on paste.debian
-
Guest36
-
Guest36
At the end the loop printing a key why it is zero when it should be something as input gone through process of squaring and multiplication
-
moneromooo
-
moneromooo
I get 8c5a513f9c6933ecbfcc8a3cbad226b8d40cf12c78b1a15a815bbd0e3fe845ce for that last key.
-
Guest36
The compiled result i get is
paste.debian.net/1248724
-
Guest36
Okay wait a minute i am trying wht you tried
-
moneromooo
The huge gobs of ubsan warnigns are known for that cod.
-
moneromooo
For me it's "don't touch or you'll break it" :D
-
Guest36
Don't touch or break it 🤣 it
-
Guest36
But it gave me zeroes again
-
Guest36
I copy paste you main code and than compiled the result is
paste.debian.net/hidden/685a3114
-
Guest36
But when I copy pasted your command to compile it gave me result can you plz explain why was special in gcc -fsanitize=address -g -O2 -I contrib//epee/include/ -I src/crypto -W -Wall ~/QubesIncoming/disp4681/test-yrt.c build/Linux/cc/release/src/crypto/libcncrypto.a
-
Guest36
That it gave result.
-
Guest36
moneromooo: thanks a lot for giving your time and helping solve my doubt it gave result but i am not getting why was so special in you compiling command that it gave result
-
Guest36
The only odd thing i not included was that libcncrypt.a but why including it gave results.
-
-
willshu[m]
Hello all, I use monero-javascript to sign keyImages offline. But this line throws an error: ` exportKeyImages Error: TypeError: (intermediate value) is not iterable`.
-
-
willshu[m]
I guess the error comes from the red box in the second image above, for ... of null will throw an error: `TypeError: (...) is not iterable.`
-
moneromooo
yrt: I don't know...
-
yrt
But you did it so how
-
moneromooo
Maybe your copy of the cncrypto functions is somehow different from the one generated in libcncrypto for some reason.
-
willshu[m]
Since this js function actually calls a `c++` function, I tried to `log/print` it.
-
-
yrt
But the file crypto-ops was self sufficient it not needed anything except it header file
-
willshu[m]
But it does not work. Where in the `monero-javascript` repo does `emscripten` translate c++ to javascript? I suppose the js function does not actually call `c++`, but instead calls js script translated from `c++`, right?
-
yrt
There should have been any errors whole exporting key images i think so
-
moneromooo
Your copy does not include some constants. Maybe your linker just links a dummy copy full of zeroes...
-
moneromooo
I needed to link the lib in for those. Like fe_ma, etc. Thy're in crypto-ops-data.c
-
Guest75
Which is best private way to first time buy monero or send monero into wallet compiled by source code
-
Guest75
Or exchange your country money into monero and send it to wallet
-
moneromooo
People in #monero or #monero-markets should know.
-
rbrunner
I checked a bit how much memory `monero-wallet-rpc` uses, on Linux using pmap which seems to work.
-
rbrunner
With a wallet open with maybe 50 outputs, it used 452796K
-
rbrunner
As soon as I mined a few blocks to it, and it made a refresh, memory usage jumped to 1168116K
-
rbrunner
Any idea why? And how would one approach to find out where 600 MB are needed to just scan a couple of blocks and receiving coinbase txs?
-
rbrunner
The block themselves are empty blocks from an offline daemon
-
plowsof
the other day i had 30 rpc wallets syncing , each using 2 GB of ram
-
rbrunner
Sporty!
-
rbrunner
I never saw anything in Monero working with other things like dynamically growing std arrays, maps, vectors. Why this suddenly gobbles RAM like crazy is beyond me right now
-
rbrunner
*other things than
-
Guest52
Hi, how are monero accounts saved? Will the accounts in my wallet be preserved when I upgrade versions to 0.18? Thank you in advance
-
gingeropolous
Guest52, better for #monero, but yes. the account in your wallet will be preserved when you upgrade. Your wallet is a key. Your monero are on the blockchain. You use your key to access the monero on the blockchain.
-
Guest35
I want to test monero source code and network how it works like it works in network but in private network like ethereum provides do i can do same in monero
-
rbrunner
Basically start a daemon with the option --offline, make it mine blocks, and direct wallet, RPC etc. to that daemon
-
rbrunner
You can influence mining using the startup option --fixed-difficulty
-
rbrunner
I use to take the current testnet blockchain as a starting point for any such experiments
-
jordanunstoppabl
Hey everyone
-
jordanunstoppabl
who can I speak with about monero wallet partnerships?
-
jordanunstoppabl
reaching out from Unstoppable Domains
-
ofrnxmr[m]
Are you actually from UD?
-
ofrnxmr[m]
Its already implemented in cake and monerujo
-
ofrnxmr[m]
(I think it works in monerujo.. might only be openalias)
-
jordanunstoppabl
yes actually from UD, will check cake and monerujo, thank you
-
ofrnxmr[m]
Ok. #monero-gui:monero.social for official gui channel
-
jordanunstoppabl
ok yeah I think that's what im looking for, thanks
-
Rucknium[m]
jordan.unstoppable.domains: Ask in #monero-community-dev:monero.social too
-
ofrnxmr[m]
#feather:monero.social for feather wallet
-
gingeropolous
dunno if y'all see this in mrl, but someone is now spamming chain tx_extra with #mrl <tx_extra[m]> follow #monero-research-lab directly from your daemon or visit mrlonchain.net
-
rbrunner
Easily winning the contest of "the most stupid thing in Monero July 2022"
-
hyc
was that serious? I didn't look at the site
-
rbrunner
Sadly, yes.
-
BusyBoredom[m]
Hilarious way to prove his point though.
-
UkoeHB
there is no real way to stop that kind of thing except massively increasing fees
-
rbrunner
Yeah, bonus point for creativity.
-
selsta
is it possible to prune tx_extra (unless it's a paymentid) ?
-
gingeropolous
i thought tx_extra had a limit on size
-
rbrunner
Even if, you don't write 5000 character lines in #MRL, do you?
-
UkoeHB
gingeropolous: like 100kB or something?
-
rbrunner
I wonder how long it will take until they take this channel instead.
-
rbrunner
Or in addition.
-
hyc
if he's logging from the IRC side and not the matrix side, it's gonna be spammed with all the re-edits
-
merope
I kicked it from the matrix side, someone on IRC please check if it's there too
-
kayabanerve[m]
<rbrunner> "Easily winning the contest of "..." <- Yep
-
kayabanerve[m]
<BusyBoredom[m]> "Hilarious way to prove his point..." <- Except we can't act on it until the next hf
-
hyc
still have 2 days left in July
-
kayabanerve[m]
We could change relay policy in 0.18.0.1?
-
UkoeHB
it is possible to inflate the fee rate of memo bytes compared to tx bytes, since tx bytes are a kind of fixed cost to adding a memo (or steganography)
-
kayabanerve[m]
If we can ban relaying > 255 bytes and charge a 2x fee, great
-
kayabanerve[m]
That may be possible as solely fee policy
-
moneromooo
FWIW, for the people concerned with bloat, it is as cheap to spam with txes, and those incur extra verification time, while tx_extra is ~free there.
-
UkoeHB
e.g. if the base tx has 10% space for steganography, that means the per-memo-byte cost is 10x the overall fee/byte; this can be extended to all other memo bytes so the weighting is 10x on each additional memo byte
-
kayabanerve[m]
moneromooo: And there's the whole negotiating with terrorists thing
-
moneromooo
So allowing spam via tx_extra is actually a good thing (unless we prevent spam via txes, and I don't think we can).
-
moneromooo
er... wat
-
kayabanerve[m]
Like I didn't mind them joining and pointing out the bee movie script lol. It was great
-
kayabanerve[m]
But we shouldn't rush dev decisions based on harassment
-
rbrunner
Exactly.
-
rbrunner
That would be a much too easy win.
-
UkoeHB
in that way there is no additional incentive to make more txs for steganography vs adding more plain memo bytes, while maximizing per/byte cost on memo stuff
-
kayabanerve[m]
I'm saying that this isn't the end of the world regarding spam. This, as their attempt to say it can be used for spam, is a fear tactic.
-
kayabanerve[m]
Which is agreeing with you mooo in a practical sense, and highlighting how this is bs
-
kayabanerve[m]
2x relay fee with a 255 byte limit is my advocacy, both short term and long term, unless we want to encourage uniformity via steganography for the theoretical benefits
-
kayabanerve[m]
We can theoretically do that not as a hard fork, if it's solely applied on mempool relay. It's fee calculation and a bool
-
UkoeHB
I disagree about a byte limit, but do think a tx weighting that amplifies memos and any steganography space would be worthwhile
-
kayabanerve[m]
Though considering we're not at capacity, we don't really have a fee market. This doesn't prove anything .-.
-
kayabanerve[m]
UkoeHB: I don't think we have legitimate use for anything greater and I do agree we shouldn't be an arbitrary file host
-
kayabanerve[m]
If we did allow it uncapped, I wonder if we should consider exponential fees. The issue is when we encourage steg :/
-
blackpenguin
hello, just found the onion mirror still gives 0.17.3.2 on download link for linux64 - and it's even worse the file received doesn't even match checksum of 0.17.3.2 beside that it should be 0.18.0.0 - hope someone around knows where to forwards this info to get it fixed
-
kayabanerve[m]
And I'm saying 255 so it fits in a single byte. While we can't save a byte without a hard fork, we can set it as policy now so we can in the future without again changing the size limit
-
moneromooo
binaryFate: ^
-
UkoeHB
amplifying memo byte weighting is equivalent to a byte limit, without adding any design constraints to the API
-
selsta
blackpenguin: which hash do you get?
-
UkoeHB
as I have said many times, the memo field is important wiggle room that reduces long-term dependency on the core team
-
kayabanerve[m]
UkoeHB: How so?
-
kayabanerve[m]
Sorry, pointless reply, I apologize to irc
-
UkoeHB
it's literally the 'anything goes' field i.e. anything we can't forsee or anything that a byte limit would require a hard fork to enable
-
UkoeHB
adding a byte limit means core suddenly has the power to assess proposed uses of monero (a certain subset)
-
rbrunner
That is probably what infuriates me most with this stunt: The idea that the world is so incredibly simple: "Can be used for spam -> must go, period."
-
UkoeHB
ArticMine: can you look at this idea of memo weighting? (we'd also need to include tx outputs in this bonus weighting, since those are the main vector for steganography)
-
kayabanerve[m]
... wait, I'm sorry, I also continually forget we have wallet data when I make these suggestions
-
kayabanerve[m]
I'm the idiot who almost just banned us to only 8 out txs
-
kayabanerve[m]
Seraphis will make it truly arb yet it isn't arb yet
-
moneromooo
...Architecture review board ?
-
kayabanerve[m]
My new proposal is a base byte size with the standard fee, a 256 byte 2x fee, and from there exponential, but I'm sure ArticMine can do better
-
blackpenguin
selsta: ad4b4be60548cddcade3cf8874579256805559d61a68e6102e4dde71284a2039 monero-monero-gui-linux-x64-v0.17.3.2.tar.bz2
-
UkoeHB
kayabanerve[m]: exponential fee would ultimately encourage tx spam, which has a verification and scanning cost
-
moneromooo
re exponential: must be done via integer math (or very well thought out bit exact floating point).
-
kayabanerve[m]
Basically, soft arbitrary limit of 256 bytes yet fully allowing larger data. I'm not going to comment on the exponent at this time though, as I'm sure a few curves can be argued for
-
kayabanerve[m]
*255
-
selsta
blackpenguin: hash matches with v0.17.3.2 gui
-
moneromooo
I guess it can be a LUT if the max is 255 bytes.
-
kayabanerve[m]
UkoeHB: Right, to steg, yet there's a computational cost and time delay, with the inefficiencies create higher fees already
-
kayabanerve[m]
The discussion has to be 255 on top of wallet data if we set a cap. While I don't completely understand koe, I respect their opinion. We'd either want a very low exponent so it's not encouraging steg or to keep it linear at 2x for anything over wallet data
-
blackpenguin
selsta: ouch. guess that's bad news for me as it doesn't match an older download I had weeks ago. but good news as at least it's not a modified version then. still the onion mirror should provide 0.18.0.0 instead
-
selsta
are you sure you didn't confuse CLI with GUI?
-
ofrnxmr[m]
selsta: I can confirm hashes match
-
ofrnxmr[m]
But
-
ofrnxmr[m]
The downloads all point to 17.3.2 and the source code zip 404's
-
ofrnxmr[m]
-
UkoeHB
kayabanerve[m]: I am saying, when you compute a tx weight, multiply the base weight of any memo + steganography bytes by the ration `'base non-memo/non-steganography bytes for a 2-in/2-out tx' / 'base memo/steganography bytes'`
-
UkoeHB
ratio*
-
UkoeHB
er it would be base weight not base bytes *
-
kayabanerve[m]
We can't determine steg bytes :/
-
kayabanerve[m]
We can only determine extra included data which isn't steg
-
kayabanerve[m]
Unless you assume all additional outputs are steg but that's also problematic
-
UkoeHB
it doesn't have to be precise
-
UkoeHB
* the weight would apply to any additional bytes added past the base amount; the goal is for the cost of 2 txs with steganography to be equal to the cost of 1 tx + adding the same amount of equivalent memo/steg bytes
-
UkoeHB
that way the base tx cost doesn't change (which is how the fee system is calibrated)
-
UkoeHB
the ratio might be only 2x tbh, since you can use all of the CLSAG decoy responses for steganography
-
gingeropolous
"<UkoeHB> it's literally the 'anything goes' field i.e. anything we can't forsee or anything that a byte limit would require a hard fork to enable" >>> i thought this concern could be addressed by keeping an "anything goes field" in the block header. Also, i think we can refer to it as AGF
-
rbrunner
No, if we ever have a need to "fix" something in any way, or make a controversial extension, it may well be per tx, not merely per block
-
gingeropolous
u can tie data in the header to a tx maybe? then it just becomes a matter of relay rules
-
rbrunner
Yeah, of course you can blow this up until it has the complexity of SegWit "tx parts that are not really tx parts, but still are" softfork :)
-
ArticMine
What we do now is use a a transaction and block weight for fee pricing, medians and penalty. This is higher than the actual transaction size when there is more than 2 outputs.
-
ArticMine
This was implemented because verification time scales as the number of outputs but actual transaction size scales as the log of the number of outputs
-
ArticMine
So this is a way to price the verification time.
-
ArticMine
The weight was calculated by adding back 80% of the size savings for multiple outputs. This is the "claw back"
-
ArticMine
Once could apply an additional weight to tx-extra / memo fields
-
ArticMine
I am however not sure that pricing is the way to deal with this since to an attacker there could be considerable value in adding toxic content to the blockchain
-
Guest68
hey
-
Guest68
can I send a tx with the payment id being regular string ie not hexadecimal?
-
ArticMine
I am more inclined to restrict or eliminate altogether tx-extra / memo fields for no coinbase transactions
-
ArticMine
I would really like to see the arguments for keeping tx_extra / memo fields.
-
ArticMine
I understand there is a case for coinbase transaction for merged mining, but for non coinbase transactions what is the use case?
-
tevador
I think everything has been said in issue 6668. People want to put their stuff in tx_extra.
-
ArticMine
I will comment further on this in issue 6668