Text   |  XML   |  ReML   |   Visible Warnings:

Useless Assignment  at pcapng.c:864

No properties have been set. | edit properties
Jump to warning location ↓ warning details...
Show Events | Options

pcapng_read_simple_packet_block

(/home/sate/Testcases/c/cve/wireshark-1.2.0/wiretap/pcapng.c)expand/collapse
Show more  
 826  pcapng_read_simple_packet_block(FILE_T fh, pcapng_block_header_t *bh, pcapng_t *pn, wtapng_block_t *wblock,int *err, gchar **err_info _U_)
 827  {
 828          int bytes_read;
 829          int block_read;
 830          guint64 file_offset64;
 831          pcapng_simple_packet_block_t spb;
 832   
 833   
 834          /* "Simple Packet Block" read fixed part */
 835          errno = WTAP_ERR_CANT_READ;
 836          bytes_read = file_read(&spb, 1, sizeof spb, fh);
 837          if (bytes_read != sizeof spb) {
 838                  pcapng_debug0("pcapng_read_simple_packet_block: failed to read packet data");
 839                  *err = file_error(fh);
 840                  return 0;
 841          }
 842          block_read = bytes_read;
 843   
 844          if(pn->byte_swapped) {
 845                  wblock->data.simple_packet.packet_len   = BSWAP32(spb.packet_len);
 846          } else {
 847                  wblock->data.simple_packet.packet_len   = spb.packet_len;
 848          }
 849   
 850          wblock->data.simple_packet.cap_len = bh->block_total_length  
 851                                               - (guint32)sizeof(pcapng_simple_packet_block_t)  
 852                                               - (guint32)sizeof(bh->block_total_length);
 853   
 854          /*g_pcapng_debug1("pcapng_read_simple_packet_block: packet data: packet_len %u",
 855                            wblock->data.simple_packet.packet_len);*/
 856   
 857          /* XXX - implement other linktypes then Ethernet */
 858          /* (or even better share the code with libpcap.c) */
 859   
 860          /* Ethernet FCS length, might be overwritten by "per packet" options */
 861          ((union wtap_pseudo_header *) wblock->pseudo_header)->eth.fcs_len = pn->if_fcslen;
 862   
 863          /* "Simple Packet Block" read capture data */
 864          errno = WTAP_ERR_CANT_READ;
 865          bytes_read = file_read((guchar *) (wblock->frame_buffer), 1, wblock->data.simple_packet.cap_len, fh);
 866          if (bytes_read != (int) wblock->data.simple_packet.cap_len) {
 867                  *err = file_error(fh);
 868                  pcapng_debug1("pcapng_read_simple_packet_block: couldn't read %u bytes of captured data",  
 869                                wblock->data.simple_packet.cap_len);
 870                  if (*err == 0)
 871                          *err = WTAP_ERR_SHORT_READ;
 872                  return FALSE;
 873          }
 874          block_read += bytes_read;
 875   
 876          /* jump over potential padding bytes at end of the packet data */
 877          if( (wblock->data.simple_packet.cap_len % 4) != 0) {
 878                  file_offset64 = file_seek(fh, 4 - (wblock->data.simple_packet.cap_len % 4), SEEK_CUR, err);
 879                  if (file_offset64 <= 0) {
 880                          if (*err != 0)
 881                                  return -1;
 882                          return 0;
 883                  }
 884                  block_read += 4 - (wblock->data.simple_packet.cap_len % 4);
 885          }
 886   
 887          return block_read;
 888  }
Show more  




Change Warning 1044.29786 : Useless Assignment

Priority:
State:
Finding:
Owner:
Note: