summaryrefslogtreecommitdiffstats
path: root/stack/pan
Commit message (Collapse)AuthorAgeFilesLines
* PAN: Always allocate in bta_pan_data_buf_ind_cbackMyles Watson2018-06-081-8/+2
| | | | | | | | | | | | | | Change I63b857d031c55d3a0754e4101e330843eb422b2a caused a double free. Move the free call to pan_data_buf_ind_cb(). Free the buffer before every return in pan_data_buf_ind_cb. Bug: 74950468 Test: manual tethering test with DUT sharing its connection Change-Id: If4526f3042699581e2cdde79a362eef0f83768eb Merged-In: If4526f3042699581e2cdde79a362eef0f83768eb (cherry picked from commit 98232b084c66368234d19fafe3076bc1c0f1b578) CVE-2018-9356
* Fix changes from September 2017 ASBAndreas Blaesius2018-03-301-19/+16
| | | | | | | | | - fix formatting difference and use official 6.0.1 patches from r81 (e.g. commit 33427d54f31adaf5b9c697f5ce642fda1dc01946 and commit 7f17ba1f8e475706727df7c50bc31ffb191d1f9d don't match googles patches for 6.0.1) Change-Id: I3187d1be2bcbc896a60100eda7c42d0e7bb5131f
* Disable PAN Reverse Tethering when connection originated by the RemotePavlin Radoslavov2017-09-171-0/+36
| | | | | | | | | | | | | | * Check for valid interactions between the three PAN profile roles per Table 1 in PAN Profile v1.0 spec. * Explicitly disable connections to the local PANU if the remote is not PANU. Bug: 63145701 Test: External script Change-Id: I29a7e404ba7e4453b6a7c59148a2b3eb7395303a Merged-In: I29a7e404ba7e4453b6a7c59148a2b3eb7395303a (cherry picked from commit 9aea2c2f92dd5245f6b35d564ce8e471fec2b4ec) CVE-2017-0783
* PAN: Fix to avoid buffer corruption while rebuilding the Bnep headerNitin Shivpure2015-06-231-2/+5
| | | | | | | | | | | | | | | | | | | A case, where 3 remote device (PANU role) is connected with DUT (NAP role). the one remote (PANU role) send BNEP_COMPRESSED_ETHERNET_DEST_ONLY packet to DUT (NAP role), which is for another remote device (PANU role). So DUT (NAP role) forwards this packet to another remote device (PANU role). As same pbuf is used across the L2CAP/BNEP/PAN layer since reception from L2CAP to writing on L2CAP CID of another PANU. Sometimes buffer (p_buf) gets corrupted, when required buffer size is greater than actual buffer size, while rebuilding the Bnep header. Using bnep_write instead of bnep_writeBuf to solve the buffer corruption issue, which allocates the new buffer & append data into it. This patch also add TRC_BNEP & TRC_PAN into conf file. Bug: 22028876 Change-Id: I125cab6d3da0f8126605bc7e9f764a27ee72fb1f
* Merge commit 'f7f839985b3931682363d2ef3b7c5cae55a842ee' into mergeEtan Cohen2015-03-313-29/+2
|\ | | | | | | Change-Id: Iaaec1ea0bf3009b7e32a9a60f697631a3f56e889
| * Collapse BNEP #defines that don't need to be configurable.Sharvil Nanavati2015-03-162-19/+0
| | | | | | | | | | | | | | | | | | | | | | In some instances, defining these macros to FALSE would result in broken runtime behavior (e.g. UUID fields containing uninitialized values). This change collapses the following defines: BNEP_SUPPORTS_PROT_FILTERS BNEP_SUPPORTS_MULTI_FILTERS BNEP_SUPPORTS_DEBUG_DUMP BNEP_SUPPORTS_ALL_UUID_LENGTHS
| * Make build more strict by adding more warning flags.Sharvil Nanavati2015-03-161-6/+0
| | | | | | | | | | | | This change introduces -Wunused-but-set-variable to help catch programming errors. It also undefines NDEBUG so assertions are fatal and defines LOG_NDEBUG=1 to suppress LOGV.
| * Remove duplicate local bdaddr management functionsZach Johnson2015-03-161-1/+0
| |
| * Removing unnecessary BTAPI #defines.June R. Tate-Gans2015-03-161-3/+2
| |
* | Cleanups needed to make clang happyBernhard Rosenkränzer2014-12-051-1/+0
|/ | | | | | | | | This patch removes a slew of unused variables and some unused functions, fixes some incomplete initializations of structs and an sprintf parameter mismatching the format string. Change-Id: I35f22fdccc8350f885cd357d3685b869224fa433 Signed-off-by: Bernhard Rosenkränzer <Bernhard.Rosenkranzer@linaro.org>
* am d6e1b1ee: Logging cleanup: HIDH, BNEP, and PAN.Sharvil Nanavati2014-06-283-68/+68
|\ | | | | | | | | * commit 'd6e1b1ee17959461a80936f7308d8129615315d2': Logging cleanup: HIDH, BNEP, and PAN.
| * Logging cleanup: HIDH, BNEP, and PAN.Sharvil Nanavati2014-06-193-68/+68
| | | | | | | | Change-Id: I3b81b6875c764f0642be4f4d8905aa24524ea902
* | am 2d5d56ea: Rewrite PAN_Write in terms of PAN_WriteBuf.Sharvil Nanavati2014-06-281-85/+32
|\| | | | | | | | | * commit '2d5d56ead7d9dd811cf9c5ef26b16c4a0b149ade': Rewrite PAN_Write in terms of PAN_WriteBuf.
| * Rewrite PAN_Write in terms of PAN_WriteBuf.Sharvil Nanavati2014-04-181-85/+32
| | | | | | | | | | | | | | | | The original code looked copy/pasted. The new implementation still results in the same number of buffer copies but is much easier to read and reduces redundancy. Change-Id: I3ace395d2b124074e9687d13ae9a1b0b93a25a31
* | am 45e9aaca: Merge "Fix bug in PAN code where no new PAN connections can be ↵Matthew Xie2014-03-101-1/+1
|\ \ | |/ |/| | | | | | | | | made." * commit '45e9aaca1c4e49f22cc23105aed6c625ad0d1278': Fix bug in PAN code where no new PAN connections can be made.
| * Merge "Fix bug in PAN code where no new PAN connections can be made."Matthew Xie2014-03-101-1/+1
| |\
| | * Fix bug in PAN code where no new PAN connections can be made.Sharvil Nanavati2014-01-311-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If you call PAN_connect() followed quickly by PAN_disconnect(), you won't be able to establish any additional PANU connections. This happens because the connection never has a chance to make it into PAN_STATE_CONNECTED and num_conns only gets decremented if you disconnect in the connected state. As a result, subsequent calls to connect will fail because the stack still believes there to be an existing connection. Change-Id: Id24873828ca4f517496cb19a44993d0e8e837a1e
| * | Various fixes for LP64Kévin PETIT2014-02-271-1/+1
| |/ | | | | | | | | | | | | | | | | | | | | | | | | | | Changes include - Cast numbers to their proper length before casting to a pointer. - Use intptr_t when pointers are stored as integers. - Eliminate 'implicit declaration' warnings Signed-off-by: Matthew Leach <matthew.leach@arm.com> Signed-off-by: Ashok Bhat <ashok.bhat@arm.com> Signed-off-by: Marcus Oakland <marcus.oakland@arm.com> Signed-off-by: Kévin PETIT <kevin.petit@arm.com> Change-Id: I5664a087c5fbc322baa9178552404dbe9a992359
* | Major warnings cleanupMike J. Chen2014-02-111-0/+2
| | | | | | | | | | | | | | | | | | | | | | | | Mostly fixing unused parameter warnings. A few other warnings also fixed like possible use of unitialized variables (no real issue found, just compiler couldn't follow the path), signed vs unsigned warning. Also fixed some typos, indent issues, removal of dead code, etc. Change-Id: I95eb887aefc4d559d7921f71a0af5f3bfb01ac01 Signed-off-by: Mike J. Chen <mjchen@google.com>
* | Fix warnings about implicit declarationsMike J. Chen2014-01-311-0/+1
|/ | | | | | | | | | | Two functions were used without including header file and generating following warnings: implicit declaration of function 'bta_sys_add_uuid' implicit declaration of function 'bta_sys_remove_uuid' Change-Id: I1c36fda7af9e838259b93bd2a29e3547d9df0eb5 Signed-off-by: Mike J. Chen <mjchen@google.com>
* Snapshot cdeccf6fdd8c2d494ea2867cb37a025bf8879bafThe Android Open Source Project2012-12-124-0/+2092
Change-Id: Ia2de32ccb97a9641462c72363b0a8c4288f4f36d