Well the preprocessor simply replaces all occurrences of the define in the code with what you specified. So I guess it will work. But _if_ GNUNET_DISK_mktemp returns an allocated string, then this leaks memory. You'd need to free the string returned by the call after it is used.
BR Martin > On 7. May 2018, at 10:03, Nils Gillmann <[email protected]> wrote: > > Nils Gillmann transcribed 3.7K bytes: >> Nils Gillmann transcribed 3.0K bytes: >>> Schanzenbach, Martin transcribed 4.1K bytes: >>>> Hey. >>>> >>>> I don't thing the "//" should be an issue (albeit unnecessary). >>> >>> Yes, you are right, I just tested it. >>> >>>> Maybe the directory is not created? >> >> Okay. I just grepped for "tmp" in the directory (src/transport) and it is >> one of the testsuites where the path is hardcoded. Earlier in this thread >> we talked about changing it, but I did not consider parts of the testsuite >> which are written in C that use /tmp... >> >> I have no time to fix it today. >> I think it's okay that the testsuite crashes occasionally, so we should just >> fix the hardcoded paths now. > > I've just started looking into our C code, and C in general, more. > In src/transport/gnunet-helper-transport-wlan-dummy.c > we define 2 fifo files (FIFO_FILE1, FIFO_FILE2). Those are currently > #define FIFO_FILE1 "/tmp/test-transport/api-wlan-p1/WLAN_FIFO_in" > #define FIFO_FILE1 "/tmp/test-transport/api-wlan-p1/WLAN_FIFO_out" > > Would > > #define FIFO_FILE1 GNUNET_DISK_mktemp("test-transport/api-wlan-p1/") > "WLAN_FIFO_in" > > work? the tests are still failing with this, but it seems like the right > directories > and files are created. > > test-suite.log (one file still had "/tmp" but in the right location, via > GNUNET_DISK_mktemp: > > ================================================= > gnunet 0.11.0: src/transport/test-suite.log > ================================================= > > # TOTAL: 55 > # PASS: 49 > # SKIP: 4 > # XFAIL: 0 > # FAIL: 2 > # XPASS: 0 > # ERROR: 0 > > .. contents:: :depth: 2 > > SKIP: test_transport_address_switch_tcp > ======================================= > > May 07 07:43:00-582453 nat-25226 WARNING upnpc failed to create port mapping > May 07 07:43:00-582555 nat-25226 WARNING upnpc failed to create port mapping > May 07 07:43:00-586782 nat-25230 WARNING upnpc failed to create port mapping > May 07 07:43:00-586836 nat-25230 WARNING upnpc failed to create port mapping > ............................May 07 07:43:28-550419 > test_transport_address_switch_tcp-25204 WARNING Testcase timed out > May 07 07:43:28-550546 test_transport_address_switch_tcp-25204 WARNING Test > did not work, as peers didn't switch (flawed testcase)! > > SKIP test_transport_address_switch_tcp (exit status: 77) > > SKIP: test_transport_address_switch_udp > ======================================= > > May 07 07:43:29-987399 transport-udp-25255 WARNING We received a sending > delay of 4289 ms for DK5W > May 07 07:43:30-312551 transport-udp-25255 WARNING We received a sending > delay of 4289 ms for DK5W > May 07 07:43:30-615122 nat-25261 WARNING upnpc failed to create port mapping > May 07 07:43:30-615809 nat-25261 WARNING upnpc failed to create port mapping > May 07 07:43:30-618851 nat-25264 WARNING upnpc failed to create port mapping > May 07 07:43:30-618928 nat-25264 WARNING upnpc failed to create port mapping > ....May 07 07:43:34-137925 transport-25255 WARNING It took us 3183 ms to send > 2600/2664 bytes to DK5W (1, udp) > .May 07 07:43:34-683575 transport-25255 WARNING It took us 3412 ms to send > 2600/2664 bytes to DK5W (1, udp) > May 07 07:43:35-115098 transport-25255 WARNING It took us 3526 ms to send > 2600/2664 bytes to DK5W (1, udp) > May 07 07:43:35-489559 transport-25255 WARNING It took us 3583 ms to send > 2600/2664 bytes to DK5W (1, udp) > .May 07 07:43:35-835496 transport-25255 WARNING It took us 3612 ms to send > 2600/2664 bytes to DK5W (1, udp) > May 07 07:43:36-167183 transport-25255 WARNING It took us 3618 ms to send > 2600/2664 bytes to DK5W (1, udp) > May 07 07:43:36-491733 transport-25255 WARNING It took us 3625 ms to send > 2600/2664 bytes to DK5W (1, udp) > .May 07 07:43:36-812649 transport-25255 WARNING It took us 3629 ms to send > 2600/2664 bytes to DK5W (1, udp) > May 07 07:43:37-131821 transport-25255 WARNING It took us 3631 ms to send > 2600/2664 bytes to DK5W (1, udp) > May 07 07:43:37-450052 transport-25255 WARNING It took us 3624 ms to send > 2600/2664 bytes to DK5W (1, udp) > .May 07 07:43:37-767875 transport-25255 WARNING It took us 3624 ms to send > 2600/2664 bytes to DK5W (1, udp) > May 07 07:43:38-085482 transport-25255 WARNING It took us 3616 ms to send > 2600/2664 bytes to DK5W (1, udp) > May 07 07:43:38-402974 transport-25255 WARNING It took us 3617 ms to send > 2600/2664 bytes to DK5W (1, udp) > .May 07 07:43:38-720397 transport-25255 WARNING It took us 3609 ms to send > 2600/2664 bytes to DK5W (1, udp) > May 07 07:43:39-037834 transport-25255 WARNING It took us 3609 ms to send > 2600/2664 bytes to DK5W (1, udp) > May 07 07:43:39-355247 transport-25255 WARNING It took us 3601 ms to send > 2600/2664 bytes to DK5W (1, udp) > .May 07 07:43:39-672687 transport-25255 WARNING It took us 3594 ms to send > 2600/2664 bytes to DK5W (1, udp) > May 07 07:43:39-990115 transport-25255 WARNING It took us 3586 ms to send > 2600/2664 bytes to DK5W (1, udp) > May 07 07:43:40-307479 transport-25255 WARNING It took us 3578 ms to send > 2600/2664 bytes to DK5W (1, udp) > May 07 07:43:40-624915 transport-25255 WARNING It took us 3570 ms to send > 2600/2664 bytes to DK5W (1, udp) > .May 07 07:43:40-942477 transport-25255 WARNING It took us 3562 ms to send > 2600/2664 bytes to DK5W (1, udp) > May 07 07:43:41-259930 transport-25255 WARNING It took us 3555 ms to send > 2600/2664 bytes to DK5W (1, udp) > May 07 07:43:41-577395 transport-25255 WARNING It took us 3547 ms to send > 2600/2664 bytes to DK5W (1, udp) > .May 07 07:43:41-894826 transport-25255 WARNING It took us 3539 ms to send > 2600/2664 bytes to DK5W (1, udp) > May 07 07:43:42-212196 transport-25255 WARNING It took us 3531 ms to send > 2600/2664 bytes to DK5W (1, udp) > May 07 07:43:42-529641 transport-25255 WARNING It took us 3523 ms to send > 2600/2664 bytes to DK5W (1, udp) > .May 07 07:43:42-847013 transport-25255 WARNING It took us 3516 ms to send > 2600/2664 bytes to DK5W (1, udp) > May 07 07:43:43-164358 transport-25255 WARNING It took us 3508 ms to send > 2600/2664 bytes to DK5W (1, udp) > May 07 07:43:43-481742 transport-25255 WARNING It took us 3500 ms to send > 2600/2664 bytes to DK5W (1, udp) > .May 07 07:43:43-799086 transport-25255 WARNING It took us 3492 ms to send > 2600/2664 bytes to DK5W (1, udp) > May 07 07:43:44-116482 transport-25255 WARNING It took us 3485 ms to send > 2600/2664 bytes to DK5W (1, udp) > May 07 07:43:44-433858 transport-25255 WARNING It took us 3469 ms to send > 2600/2664 bytes to DK5W (1, udp) > .May 07 07:43:44-751066 transport-25255 WARNING It took us 3461 ms to send > 2600/2664 bytes to DK5W (1, udp) > May 07 07:43:45-385635 transport-25255 WARNING It took us 3770 ms to send > 2600/2664 bytes to DK5W (1, udp) > May 07 07:43:45-544278 transport-25255 WARNING It took us 3604 ms to send > 2600/2664 bytes to DK5W (1, udp) > .May 07 07:43:45-782322 transport-25255 WARNING It took us 3516 ms to send > 2600/2664 bytes to DK5W (1, udp) > May 07 07:43:46-060020 transport-25255 WARNING It took us 3469 ms to send > 2600/2664 bytes to DK5W (1, udp) > May 07 07:43:46-357545 transport-25255 WARNING It took us 3441 ms to send > 2600/2664 bytes to DK5W (1, udp) > .May 07 07:43:46-665023 transport-25255 WARNING It took us 3423 ms to send > 2600/2664 bytes to DK5W (1, udp) > May 07 07:43:46-977444 transport-25255 WARNING It took us 3411 ms to send > 2600/2664 bytes to DK5W (1, udp) > May 07 07:43:47-292388 transport-25255 WARNING It took us 3401 ms to send > 2600/2664 bytes to DK5W (1, udp) > May 07 07:43:47-608397 transport-25255 WARNING It took us 3391 ms to send > 2600/2664 bytes to DK5W (1, udp) > .May 07 07:43:47-925138 transport-25255 WARNING It took us 3383 ms to send > 2600/2664 bytes to DK5W (1, udp) > May 07 07:43:48-242355 transport-25255 WARNING It took us 3375 ms to send > 2600/2664 bytes to DK5W (1, udp) > May 07 07:43:48-559630 transport-25255 WARNING It took us 3367 ms to send > 2600/2664 bytes to DK5W (1, udp) > .May 07 07:43:48-876932 transport-25255 WARNING It took us 3367 ms to send > 2600/2664 bytes to DK5W (1, udp) > May 07 07:43:49-194289 transport-25255 WARNING It took us 3359 ms to send > 2600/2664 bytes to DK5W (1, udp) > May 07 07:43:49-511676 transport-25255 WARNING It took us 3343 ms to send > 2600/2664 bytes to DK5W (1, udp) > .May 07 07:43:49-828962 transport-25255 WARNING It took us 3336 ms to send > 2600/2664 bytes to DK5W (1, udp) > May 07 07:43:50-463606 transport-25255 WARNING It took us 3645 ms to send > 2600/2664 bytes to DK5W (1, udp) > May 07 07:43:50-622286 transport-25255 WARNING It took us 3478 ms to send > 2600/2664 bytes to DK5W (1, udp) > .May 07 07:43:50-860334 transport-25255 WARNING It took us 3391 ms to send > 2600/2664 bytes to DK5W (1, udp) > May 07 07:43:51-138021 transport-25255 WARNING It took us 3344 ms to send > 2600/2664 bytes to DK5W (1, udp) > May 07 07:43:51-435536 transport-25255 WARNING It took us 3316 ms to send > 2600/2664 bytes to DK5W (1, udp) > .May 07 07:43:51-743034 transport-25255 WARNING It took us 3298 ms to send > 2600/2664 bytes to DK5W (1, udp) > May 07 07:43:52-055451 transport-25255 WARNING It took us 3286 ms to send > 2600/2664 bytes to DK5W (1, udp) > May 07 07:43:52-370367 transport-25255 WARNING It took us 3275 ms to send > 2600/2664 bytes to DK5W (1, udp) > .May 07 07:43:52-686423 transport-25255 WARNING It took us 3266 ms to send > 2600/2664 bytes to DK5W (1, udp) > May 07 07:43:53-003068 transport-25255 WARNING It took us 3258 ms to send > 2600/2664 bytes to DK5W (1, udp) > May 07 07:43:53-320191 transport-25255 WARNING It took us 3250 ms to send > 2600/2664 bytes to DK5W (1, udp) > May 07 07:43:53-637493 transport-25255 WARNING It took us 3242 ms to send > 2600/2664 bytes to DK5W (1, udp) > .May 07 07:43:53-955073 transport-25255 WARNING It took us 3242 ms to send > 2600/2664 bytes to DK5W (1, udp) > May 07 07:43:54-272558 transport-25255 WARNING It took us 3226 ms to send > 2600/2664 bytes to DK5W (1, udp) > May 07 07:43:54-589951 transport-25255 WARNING It took us 3219 ms to send > 2600/2664 bytes to DK5W (1, udp) > .May 07 07:43:54-907352 transport-25255 WARNING It took us 3211 ms to send > 2600/2664 bytes to DK5W (1, udp) > May 07 07:43:55-224748 transport-25255 WARNING It took us 3203 ms to send > 2600/2664 bytes to DK5W (1, udp) > May 07 07:43:55-542145 transport-25255 WARNING It took us 3195 ms to send > 2600/2664 bytes to DK5W (1, udp) > .May 07 07:43:55-859530 transport-25255 WARNING It took us 3188 ms to send > 2600/2664 bytes to DK5W (1, udp) > May 07 07:43:56-176951 transport-25255 WARNING It took us 3179 ms to send > 2600/2664 bytes to DK5W (1, udp) > May 07 07:43:56-494289 transport-25255 WARNING It took us 3172 ms to send > 2600/2664 bytes to DK5W (1, udp) > .May 07 07:43:56-811628 transport-25255 WARNING It took us 3156 ms to send > 2600/2664 bytes to DK5W (1, udp) > May 07 07:43:57-128992 transport-25255 WARNING It took us 3148 ms to send > 2600/2664 bytes to DK5W (1, udp) > May 07 07:43:57-446348 transport-25255 WARNING It took us 3140 ms to send > 2600/2664 bytes to DK5W (1, udp) > .May 07 07:43:57-763622 transport-25255 WARNING It took us 3132 ms to send > 2600/2664 bytes to DK5W (1, udp) > May 07 07:43:58-081010 transport-25255 WARNING It took us 3125 ms to send > 2600/2664 bytes to DK5W (1, udp) > May 07 07:43:58-398395 transport-25255 WARNING It took us 3117 ms to send > 2600/2664 bytes to DK5W (1, udp) > May 07 07:43:58-581816 test_transport_address_switch_udp-25239 WARNING > Testcase timed out > May 07 07:43:58-581934 test_transport_address_switch_udp-25239 WARNING Test > did not work, as peers didn't switch (flawed testcase)! > > SKIP test_transport_address_switch_udp (exit status: 77) > > SKIP: test_transport_address_switch_http > ======================================== > > May 07 07:44:00-650827 nat-25297 WARNING upnpc failed to create port mapping > May 07 07:44:00-653529 nat-25297 WARNING upnpc failed to create port mapping > May 07 07:44:00-654786 nat-25299 WARNING upnpc failed to create port mapping > May 07 07:44:00-655803 nat-25299 WARNING upnpc failed to create port mapping > ............................May 07 07:44:28-615255 > test_transport_address_switch_http-25274 WARNING Testcase timed out > May 07 07:44:28-615366 test_transport_address_switch_http-25274 WARNING Test > did not work, as peers didn't switch (flawed testcase)! > > SKIP test_transport_address_switch_http (exit status: 77) > > SKIP: test_transport_address_switch_https > ========================================= > > May 07 07:44:30-701806 nat-25337 WARNING upnpc failed to create port mapping > May 07 07:44:30-702063 nat-25337 WARNING upnpc failed to create port mapping > May 07 07:44:30-752393 nat-25342 WARNING upnpc failed to create port mapping > May 07 07:44:30-752491 nat-25342 WARNING upnpc failed to create port mapping > ............................May 07 07:44:58-647589 > test_transport_address_switch_https-25309 WARNING Testcase timed out > May 07 07:44:58-647706 test_transport_address_switch_https-25309 WARNING Test > did not work, as peers didn't switch (flawed testcase)! > > SKIP test_transport_address_switch_https (exit status: 77) > > FAIL: test_plugin_udp > ===================== > > May 07 07:45:08-815979 test-plugin-transport-25427 ERROR Could not copy > hostkey `test_plugin_hostkey.ecc' to destination > `/tmp/guix-build-gnunet-git1.drv-0/gnunet//test-gnunetd-plugin-transport//.local/share/gnunet//private_key.ecc'. > Exiting. > FAIL test_plugin_udp (exit status: 1) > > FAIL: test_plugin_unix > ====================== > > May 07 07:45:08-842954 test-plugin-transport-25446 ERROR Could not copy > hostkey `test_plugin_hostkey.ecc' to destination > `/tmp/guix-build-gnunet-git1.drv-0/gnunet//test-gnunetd-plugin-transport//.local/share/gnunet//private_key.ecc'. > Exiting. > FAIL test_plugin_unix (exit status: 1) > > > > But I don't think the issue is with those files, they are known to the > testsuite > and it doesn't care where the files are. There are more "/tmp" in > src/transport to fix. > > >>>> BR >>>> >>>>> On 7. May 2018, at 07:59, Nils Gillmann <[email protected]> wrote: >>>>> >>>>> Nils Gillmann transcribed 1.8K bytes: >>>>>> Christian Grothoff transcribed 3.9K bytes: >>>>>>> On 05/06/2018 08:34 PM, Nils Gillmann wrote: >>>>>>>> Christian Grothoff transcribed 2.9K bytes: >>>>>>>>> On 05/05/2018 09:58 PM, Nils Gillmann wrote: >>>>>>>>>> Anyways I just had some time to work on this, could you read >>>>>>>>>> 5f627ce1a78f662d928fa6c73a90342d8bff5cf3 and tell me if I understood >>>>>>>>>> the conversation we had here and your small commit right? >>>>>>>>> >>>>>>>>> Yes, you got it perfectly right. >>>>>>>>> >>>>>>>> >>>>>>>> I've changed the config files, now the problem for us in Nix and Guix >>>>>>>> will be that the silly maximum unix path length will be hit quiet >>>>>>>> often... >>>>>>>> at least for the Guix build chroot when the chroot is in >>>>>>>> /tmp/guix-build-gnunet-git1.drv-0/ >>>>>>>> >>>>>>>> The $GNUNET_TMP then gives you diretories like >>>>>>>> /tmp/guix-build-gnunet-git1.drv-0/gnunet/WAHTEVER-HERE/GOES/NOTHING >>>>>>>> where previously they just landed in /tmp iirc. >>>>>>>> >>>>>>>> We should make it possible to set the value of GNUNET_TMP before make >>>>>>>> check, >>>>>>>> for example via the equivalent of export GNUNET_TMP="/tmp"; make check. >>>>>>>> >>>>>>>> It's quiet unfortunate with the length.. >>>>>>> >>>>>>> Have you considered re-defining the maximum path length? Given that you >>>>>>> are a distro, you should be able to recompile everything necessary >>>>>>> (kernel, libc) with simply a larger value, right? >>>>>> >>>>>> This time it wasn't the path length. We need to fix some parts of the >>>>>> test suite to adopt to the new (dynamic) locations of the test folders. >>>>>> >>>>>> I knew this, but I was preoccupied. >>>>>> >>>>>> I can't do anything today, feel free to revert >>>>>> 9fbda6db66bf8dc074de6c8792f369801e4e5999 >>>>>> or fix the test suites that are affected. I myself can think of fixes >>>>>> tomorrow afternoon/evening. >>>>>> For now the testsuite is broken with this. >>>>> >>>>> One fixed, a couple more to go. >>>>> >>>>> >>>>> FAIL: test_plugin_unix >>>>> ====================== >>>>> >>>>> May 07 05:37:28-321787 test-plugin-transport-25451 ERROR Could not copy >>>>> hostkey `test_plugin_hostkey.ecc' to destination >>>>> `/tmp/guix-build-gnunet-git1.drv-0/gnunet//test-gnunetd-plugin-transport//.local/share/gnunet//private_key.ecc'. >>>>> Exiting. >>>>> FAIL test_plugin_unix (exit status: 1) >>>>> >>>>> >>>>> ^ should it be $GNUNET_TMP/foo or ${GNUNET_TMP}foo ? The two forward >>>>> slashes are irritating. >>>>> >>>>>> _______________________________________________ >>>>>> GNUnet-developers mailing list >>>>>> [email protected] >>>>>> https://lists.gnu.org/mailman/listinfo/gnunet-developers >>>>> >>>>> _______________________________________________ >>>>> GNUnet-developers mailing list >>>>> [email protected] >>>>> https://lists.gnu.org/mailman/listinfo/gnunet-developers >>>> >>> >>> >> >> _______________________________________________ >> GNUnet-developers mailing list >> [email protected] >> https://lists.gnu.org/mailman/listinfo/gnunet-developers
signature.asc
Description: Message signed with OpenPGP
_______________________________________________ GNUnet-developers mailing list [email protected] https://lists.gnu.org/mailman/listinfo/gnunet-developers
