search

lucaderi / sgr

Progetti Corso Gestione di Rete
https://didattica.di.unipi.it/laurea-in-informatica/insegnamenti/lista-dei-corsi/?anno=2019
14 stars 110 forks source link

progetto ferrari stanganini nmp #292

Closed StangaSimo closed 1 year ago

StangaSimo commented 1 year ago

Abbiamo aggiunto la relazione con la spiegazione del funzionamento del programma e un esempio di test, nella cartella c'è anche il codice per simulare un black hole bloccando i pacchetti del proprio pc.

lucaderi commented 1 year ago

Lo stesso vale per la dipendenza di roaring.c/.h che non e' codice vostro e quindi va referenziato

lucaderi commented 1 year ago

Qui c'e' un piccolo problema ==76147== Conditional jump or move depends on uninitialised value(s) ==76147== at 0x14F267: min (main.c:135) ==76147== by 0x14F2EB: free_entry (main.c:147) ==76147== by 0x14FED0: print_hash_entry (main.c:326) ==76147== by 0x10B193: hashmap_iterate (map.c:372) ==76147== by 0x14FF50: print_stats (main.c:337) ==76147== by 0x150768: dummyProcesssPacket (main.c:527) ==76147== by 0x48DDC28: ??? (in /usr/lib/x86_64-linux-gnu/libpcap.so.1.10.1) ==76147== by 0x48DE1C3: ??? (in /usr/lib/x86_64-linux-gnu/libpcap.so.1.10.1) ==76147== by 0x48E207D: pcap_loop (in /usr/lib/x86_64-linux-gnu/libpcap.so.1.10.1) ==76147== by 0x150A8F: main (main.c:602)

lucaderi commented 1 year ago

Qui ci sono dei leaks

==76147== HEAP SUMMARY: ==76147== in use at exit: 70,213 bytes in 863 blocks ==76147== total heap usage: 1,285 allocs, 422 frees, 883,543 bytes allocated ==76147== ==76147== LEAK SUMMARY: ==76147== definitely lost: 1,613 bytes in 146 blocks ==76147== indirectly lost: 12,964 bytes in 454 blocks ==76147== possibly lost: 0 bytes in 0 blocks ==76147== still reachable: 53,620 bytes in 242 blocks ==76147== suppressed: 0 bytes in 0 blocks ==76147== Rerun with --leak-check=full to see details of leaked memory ==76147==

lucaderi commented 1 year ago

Vedo inoltre a runtime dei log del tipo

|⬤ | 192.168.2.178 | 11.43.25 | 11.43.25 | 2035:8820 | sender: ip: 192.168.2.134 con le porte: 11400 34992 47115 47285 49845 ip: 192.168.2.153 con le porte: 5632

oppure

sender: ip: 192.168.2.134 con le porte: 225 227 740 741 1248 1251 1763 2274 2275 2786 3296 3297 3809 3811 3812 3814 4303 4324 4815 4834 4837 4838 5345 5349 5350 5859 7392 7396 7398 7906 7908 8417 8419 8932 8933 9440 9442 9958 10467 11491 12001 12006 12517 13024 13027 13028 13538 13542 14048 14049 14050 14051 14054 15072 15078 16101 16102 16613 17124 18145 18146 18147 18656 18661 19169 19174 20193 20196 21216 21217 21218 21221 22240 22243 22246 22708 22756 22757 23269 23777 23782 24288 24756 24802 24804 24806 25313 25318 25829 26339 26850 26854 27361 27872 27876 27877 27878 28386 28390 28896 29411 29412 29920 29921 29922 29924 30438 30949 31968 32993 32994 32995 33506 34017 34019 34020 34022 34533 35042 35552 35555 35556 35557 36065 36068 37090 37603 38114 38118 38624 38629 39137 39138 39649 39652 40673 41190 41696 41699 41701 42721 42722 42724 42725 42726 43744 44255 44773 45284 45286 45795 46304 46307 46310 46817 46818 46819 47115 47332 47792 48865 48868 48869 49382 49888 49891 50912 50914 50917 51428 51935 51942 52449 52452 53471 53474 53475 53990 54495 54499 54500 54501 55008 55009 55522 56544 56546 56550 57057 57505 57567 57571 58080 58085 58591 58596 58597 59108 59618 59619 60128 60129 60643 60644 61152 61155 61663 61666 61668 62178 62180 62181 62689 63711 63714 64225 64735 64736 65251 65253 ip: 192.168.2.153 con le porte: 5632

Ma non vedo queste porte se catturo il traffico contemporaneamente al vostro tool mentre vedo altre porte che non sono invece riportate. Penso ci siano dei problemi di gestione delle porte.

caro023 commented 1 year ago

Abbiamo i leak di memoria presenti nelle strutture dati. Ci sono ancora dei leak ma sono dati dalla libreria rrd.h. Se usiamo la funzione system chiamando direttamente rrdtool risolviamo questi problemi, quale è secondo lei la soluzione migliore?

lucaderi commented 1 year ago

Fissate i vostri problemi e fate si che il programma produca l'output atteso. Se ci sono leaks in codice che non e' il vostro lasciate pure (anche se mi pare strano). In attesa di fix per i problemi sopra.

StangaSimo commented 1 year ago

Abbiamo fatto 3 commit per i fix dei problemi da lei elencati:

Nel [c007693] ci siamo sbagliati e manca una funzione del patriciatree.c, provvedo a sistemare il problema il prima possibile

lucaderi commented 1 year ago

I problemi segnalati sono stati risolti anche se il disegno dell'applicazione si potrebbe semplificare e ridurre la memoria complessiva.