Bump version to 3.4

Version 3.4 fixes a race condition where two threads could enter the
check_next_chunk() function simultaneously and end up with mismatched
chunk numbers.

Change-Id: I348f5cf11d9826e4f8df20b02d256f3684ad2318
Signed-off-by: Dongwoo Lee <dwoo08.lee@samsung.com>

net: Make check_next_chunk() mutually exclusive

Since 4004285311 ("Fix the race in data_finished() and resp_finished()"),
Although the possibility of a race between two threads executing
data_finished() and resp_finished() is reduced, it is still possible
to enter the check_next_chunk() function at the same time in certain
circumstances as below:

Thread A  | Thread B
-------------------------------------------------+--------------------------------------------------------------
 | chunk->resp_finished = 1;
chunk->data_finished = 1;  |
if (chunk->resp_finished)      |
check_next_chunk(chunk, transfer);  |
chunk->chunk_number = transfer->chunk_number++;  |
 | if (chunk->data_finished)
 | check_next_chunk(chunk, transfer);
 |      >> chunk->chunk_number = transfer->chunk_number++;
                                                 | <chunk_number becomes unmatched>
chunk->data_finished = chunk->resp_finished = 0; |
…

To address this, this makes the functions completely mutually exclusive.

Change-Id: I94df217ec82dadc322050693e86763f9c8213d05
Signed-off-by: Dongwoo Lee <dwoo08.lee@samsung.com>

Bump version to 3.3

In version 3.3 lthor, issue for thor sent file size less
than 1KB is fixed

Change-Id: I18b64284327e0e9393bf22cabe1873c13aafd72b
Signed-off-by: Seung-Woo Kim <sw0312.kim@samsung.com>

Fix report_progress when sent file size is less than 1KB

When sent file is less than 1KB, there is a divide-by zero error.
Check size less than 1KB to fix the issue.

Change-Id: Ie73fa6e9151f635a6420153c025654462238498c
Signed-off-by: Seung-Woo Kim <sw0312.kim@samsung.com>

Fix the race in data_finished() and resp_finished()

Setting the data_finished and resp_finished flags too early sometimes
led to a situation in which another chunk was sent, even though the
functions had not finished executing, which resulted in a flashing
error.

For example:
When data_finished() was called, the flag data_finished was set. Then
after entering aio_error/aio_return the thread would be put to sleep and
the program would start executing the resp_finished() function. The
function was executed to the end and because data_finished flag was set,
the check_next_chunk() was called.
After that, the thread with data_finished() would return from sleep and
continue its execution, even though the data it was pointing to at that
moment is out of date. This led, among other things, to a mismatch
between numbers of chunk and the response.

Change-Id: I404b3723cb9e671d13221bff2b7823f1fdc43906
Signed-off-by: Mateusz Moscicki <m.moscicki2@samsung.com>

Bump version to 3.2 for Ubuntu 18.04 build

In version 3.2 build issue in Ubuntu 18.04 is resolved and build
warnings are also removed.

Change-Id: Idc4973bfea3b018095bb13f4445ac0307d354288
Signed-off-by: Seung-Woo Kim <sw0312.kim@samsung.com>

net: add include for signal.h

To use aio_sigevent in Ubuntu 18.04, add explict include for
signal.h.

Change-Id: Icbf15f411aa1466879dc017c13311d408ab18a96
Signed-off-by: Seung-Woo Kim <sw0312.kim@samsung.com>

transport: remove implicit-function-declaration build warnings

Remove implicit-function-declaration build warnings by adding
function declarations to transport header.

Change-Id: Id258ab568c3eba49b1e443734443f9b207b86268
Signed-off-by: Seung-Woo Kim <sw0312.kim@samsung.com>

Bump version to 3.1 fixing crash in network mode

In version 3.0, -o0 optimization has segment falut crash and it
is fixed. To show the modification clearly, bump version to 3.1.

Change-Id: I0ef174d2a780007248f59f07669055d7b40c0ebb
Signed-off-by: Seung-Woo Kim <sw0312.kim@samsung.com>

Fix segmentation fault during network tranfer

Since 'chunk' which is unit of transfer is not initialized, aio_read()
is failed with invalid argument error. Moreover, in this case,
chunk.nh is even NULL, so chunk->nh->sock_fd cause segmentation fault.
Thus, this patch fixes to initialize chunk properly, and make chunk.nh
have network handler pointer.

Change-Id: I322a01ca99829eb928f26488a341cdaee1c4d9e2
Signed-off-by: Dongwoo Lee <dwoo08.lee@samsung.com>

Bump version to 3.0 supporting THOR protocol 5.0 and network mode

The lthor 3.0 supports THOR protocol 5.0 which can send file more
than 4GB and supports flashing via network.

Change-Id: I7c8429d01488281b2ae4c59f7846da78ee1a80e7
Signed-off-by: Seung-Woo Kim <sw0312.kim@samsung.com>

Add support for asynchronous transfer on network transport

To improve performance of network transfer, add asynchronous
transfer with AIO APIs to network transport.

Change-Id: I91dbf1161aeebc03af2bccdbe4ed0e20a322949d
Signed-off-by: Dongwoo Lee <dwoo08.lee@samsung.com>
Signed-off-by: Seung-Woo Kim <sw0312.kim@samsung.com>

Add network transport for image transfer

This patch adds network transport which can transfer image via tcp
layer. If the option "--net=<ip[:port]>" is passed, lthor tries to
connect the target with given ip and port, or else, it just try to
connect via usb.

When the port nubmer is not provided for network option, it uses
default port.

Change-Id: I14bea3523d01e3a4f547e41d40d353790a870460
Signed-off-by: Dongwoo Lee <dwoo08.lee@samsung.com>
Signed-off-by: Seung-Woo Kim <sw0312.kim@samsung.com>

Re-structure lthor to extend its transfer methods

Currently, thor has very specific code structure for considering only
usb transfer, thus it is too hard to add other transfer methods. This
patch makes lthor to be extendable by just implementing few operations:
open, close, send, recv, and send_data.

Change-Id: If455a9d7a5c7fd86cf6e8808f29c643456e80166
Signed-off-by: Dongwoo Lee <dwoo08.lee@samsung.com>
Signed-off-by: Seung-Woo Kim <sw0312.kim@samsung.com>

Add packed attribute to message protocol struct

The structures for message should be packed for communication. Add
explict packed attribute to the message struct.

Change-Id: I33ec2986a5caedf0dfa248b2e04c51ee75ae1520
Signed-off-by: Seung-Woo Kim <sw0312.kim@samsung.com>

Fix wrong opt usage for pid

The opt '--product-id' should be set in pid, but it is set to vid.
Fix wrong opt usage for pid.

Change-Id: I360cacdade6dd81cd3250013c68a561ebfbdc008
Signed-off-by: Seung-Woo Kim <sw0312.kim@samsung.com>

Fix to check protocol version before refusing large file

From protocol version 5.0, it will be supported large file size
with 64bit value. Fix to check protocol version before resuing
large file.

Change-Id: Ie2b76d5e857c143289332e543bb65254ac522081
Signed-off-by: Seung-Woo Kim <sw0312.kim@samsung.com>

Update for THOR 5.0 support

Until THOR 4.0, it uses only 32bit to send file size. From THOR
5.0, it starts to use two 32bit variables for the file size.
Update for the THOR 5.0 protocol support.

Change-Id: I3569d51179bb75b034a62f902e8640a1232d6ed3
Signed-off-by: Seung-Woo Kim <sw0312.kim@samsung.com>

Fix overflow of filename

In thor protocol, only filename with 32 characters is allowed to
send but there was overflow of filename more than 32 and it is
crashed with 193 or longer length filename. Fix the overflow of
filename.

Note: ARTIK530 and ARTIK710 already misuse overflowed 33 length dtb
filename, and exact modification as thor protocol breaks flashing
on the boards. So fix with extention for the ARTIK borads by
appending more 32 bytes in rqt_pkt.str_data[1] as artik u-boot
expects.

Change-Id: I4081024cafa276617497091e00cbb6fbc21348b7
Signed-off-by: Seung-Woo Kim <sw0312.kim@samsung.com>

Fix wrong close for negative file descriptor

There is an issue which negative file descriptor can be attempt to
close in static analysis tool result. This patch fixes to skip close
on that case.

Change-Id: Ic68803bc96ab6e3737d1bcc4779b8e3c58c7ef71
Signed-off-by: Seung-Woo Kim <sw0312.kim@samsung.com>
Signed-off-by: Dongwoo Lee <dwoo08.lee@samsung.com>

Fix uninitialized data in error ret value

There is uninitialized data issue in static analysis tool result.
Fix the uninitialized data in error ret value.

Change-Id: I7e5e0f6fa372c125c5ce6eb8df321cb08919b9cf
Signed-off-by: Seung-Woo Kim <sw0312.kim@samsung.com>

Fix use of deprecated libarchive function calls

To get rid of build warning about deprecated funcations of libarchive,
this patch replaces those function calls with alternative ones.

Change-Id: I164c60243e20fcbbdad2185a14534fe7a4daf9d8
Signed-off-by: Dongwoo Lee <dwoo08.lee@samsung.com>

Fix progress report for larger than 2GB file

Currently, progress report function has 32bit signed value for its
send and left parameters, so it causes negative value progress. This
fixes the parameters as unsigned 32bit type to report properly.

Change-Id: I1d291c1507170c57b025d4fbd91098b48a97cdd0
Signed-off-by: Seung-Woo Kim <sw0312.kim@samsung.com>

Fix description for short version of params

Change-Id: I8a06e7c231f1134d2892dade182fec2b313bc630
Reported-by: Donghoon Shin <dhs.shin@samsung.com>
Signed-off-by: Krzysztof Opasiak <k.opasiak@samsung.com>

Fix busid param parsing

Change-Id: Ia3e6047ccd9f07d1a834c4eb58408d0afca1c8c3
Signed-off-by: Krzysztof Opasiak <k.opasiak@samsung.com>

Fix memleak of entry name

Change-Id: I57fe94546759888697871fa716e865771801b00b
Signed-off-by: Krzysztof Opasiak <k.opasiak@samsung.com>

Fix file testing

Change-Id: Ide8250910005c37a6ecadcb81d4d3b8e1e646bda
Reported-by: Tomasz Wozniak <t.wozniak@samsung.com>
Signed-off-by: Krzysztof Opasiak <k.opasiak@samsung.com>

Fix double free on error condition

Change-Id: I41224563de4d6176e60a5f4c893560e3dcf96b86
Reported-by: Krzysztof Kozlowski <k.kozlowski.k@gmail.com>
Signed-off-by: Krzysztof Opasiak <k.opasiak@samsung.com>

Fix memleak of entries array

Change-Id: I39259e1bdd48fc2a75cdb6652e918cef2775774c
Signed-off-by: Krzysztof Opasiak <k.opasiak@samsung.com>

Update package build configuration

Based on initial work by Seung-Woo Kim <sw0312.kim@samsung.com>.

Change-Id: I8eaa2e5d726aff359de221d1bce203439bcacd05
Signed-off-by: Krzysztof Opasiak <k.opasiak@samsung.com>

Bump lthor version to 2.0

Due to major changes in lthor implementation let's bump
version to 2.0

Change-Id: I7b39080031c499a7e9f0899b32918d682d5f7497
Signed-off-by: Krzysztof Opasiak <k.opasiak@samsung.com>

Allow to print also content of tar file

Old implementation of lthor was printing not only name of
tar file and it's size but also name of each entry and it's
size. This patch allows also new implementation to do this.

Change-Id: I05c12fb5368bcaaaeeb93a3116d9322ecc4349b1
Signed-off-by: Krzysztof Opasiak <k.opasiak@samsung.com>

Add support for large files

Currently session size is send over USB as integer 32b.
So max size of all files is 2GB.
Some fixed uboot versions treat this value as unsigned integer 32b.
So there max size of all files is 4GB.

Let's provide initial support for large files in lthor by
counting all size using off_t which thanks to define is 64b
on all platforms.

This patch also add a suitable warning if image is larger than
2GB and reports an error if image is larger than 4GB.

Change-Id: I402b3684fa96c7531edb580e38f109ecc8a5ebd0
Signed-off-by: Krzysztof Opasiak <k.opasiak@samsung.com>

Improve colors handling in terminal

Instead of repeating magick strings in many places
let's use a suitable defines for them to make the code
more readable.

Change-Id: I7b4935bd459e375be39d1dd223d20c71635670cc
Signed-off-by: Krzysztof Opasiak <k.opasiak@samsung.com>

Use asynchronous API to improve performance

Change-Id: I18f0aa14778d06751e7005e4b555c6c68f2839d4
Signed-off-by: Krzysztof Opasiak <k.opasiak@samsung.com>

Start using new implementation of thor protocol

Drop all legacy stuff which used cdc-acm kernel driver
and start using new implementation provided by libthor.

Change-Id: If3ee224b4905778d39c8b049f38353f499cd9f9f
Signed-off-by: Krzysztof Opasiak <k.opasiak@samsung.com>

Add core file with thor communication protocol

Add main file of libthor with implementation of API and thor
communication protocol.

Change-Id: Ic6e4c13ad89eef8080f955af5a71c11cd201ad78
Signed-off-by: Krzysztof Opasiak <k.opasiak@samsung.com>

Add cdc-acm related implementation of thor protocol

Thor protocol "emulates" cdc-acm class. This commit adds
some helper functions which are required to properly
setup communication on cdc-acm protocol level.

Change-Id: Iffb79c813f97bc3f3c4260f6fd713d0ea4d99353
Signed-off-by: Krzysztof Opasiak <k.opasiak@samsung.com>

Add libusb backend for thor protocol

Add implementation of functions used for communication
with tharget on USB level via libusb.

Change-Id: I59603b3bb05570e8f13c8ca3e4f48fabf2fc03c0
Signed-off-by: Krzysztof Opasiak <k.opasiak@samsung.com>

Add code for accessing tar archives

Add implementation of data source for accessing tar files.
This is needed to flash tar.gz file using lthor.

Change-Id: Id5abbf258d41f37c569c65753bb8866bb9a84b6a
Signed-off-by: Krzysztof Opasiak <k.opasiak@samsung.com>

Add code for accessing raw file by liblthor

Implement a data source for accessing standard file.
This type of access is used for reading PIT file.

Change-Id: Icc9de3718e1234e0758c27f4721f3072d3d538f9
Signed-off-by: Krzysztof Opasiak <k.opasiak@samsung.com>

Add headers with API for new implementation of thor protocol

Add new headers with:
- API for accessing new implementation of thor protocol
- Internal data structure which will be used by libthor

Copy also header with definitions of messages used by thor
protocol.

Change-Id: I0e6901a45737e3711f9328f757f76de96b88bdd3
Signed-off-by: Krzysztof Opasiak <k.opasiak@samsung.com>

Use getopt_long() to parse cmd line

Use a standard linux way of parsing cmd line arguments
to avoid unnecessary code.

Change-Id: I64325e7804048c1abce013f78df5163ee4af85c8
Signed-off-by: Krzysztof Opasiak <k.opasiak@samsung.com>

Add gitignore

Change-Id: I4126d7a25a29414dc92f776da7c2d34c731e2fa9
Signed-off-by: Krzysztof Opasiak <k.opasiak@samsung.com>

Update lthor version

This patch update lthor version to 1.5

Change-Id: Ie25e95b6fd5e4a601c64bdf8c4a3d66f0ed64005
Signed-off-by: Jaewon Kim <jaewon02.kim@samsung.com>

Add to pass abnormal file

When trying abnormal file(not matched with pit) download,
lthor pass this file.

Change-Id: Ib3c46fdd2f82a4299dfbfcaa9ad64f4b30accb56
Signed-off-by: Jaewon Kim <jaewon02.kim@samsung.com>

Add ttyACM check for lthor

This patch add ttyACM check to find correct port for lthor
even if there are several ttyACM port.

Change-Id: Icb87a9ebbabec2e5fa7c3312efff972488dbc3d1
Signed-off-by: Jaewon Kim <jaewon02.kim@samsung.com>

Remove vendor and product match code

This patch remove vendor and product ID checking code
to support various boards

Change-Id: I6e1bca14e504a072f93d661666a9c623802ef472
Signed-off-by: Jaewon Kim <jaewon02.kim@samsung.com>

Update URL in spec file

Change-Id: I6913146ba412e2514b0e99c50908a2441f4c9fd0

Correcting usage help for check protocol

Change-Id: If17ad57e29b50f01011c3435c81fb8f522eaa0b4

Increased version in debian/changelog

Change-Id: Ic49555ddc4d72ed837e78b6a053293c412394a34
Signed-off-by: Ed Bartosh <eduard.bartosh@intel.com>

Support MAC OS X 10.7,10.8

On OS X the serial device must be opened with O_NONBLOCK flag.
Later this flag must be cleared before write operations otherwise
only small chunks data can be written at once.
Tested on OS X (10.8.4).

Author : Piotr Wilczek <p.wilczek@samsung.com>

Change-Id: I2b77cf16aacd491af9c94203fe7dfc49ce361516
Signed-off-by: jaehoon.you <jaehoon.you@samsung.com>

Fixed OpenSUSE 1.23 build

Change-Id: I462628a57597e2cc8892bd705fdbf426361cf024
Signed-off-by: Ed Bartosh <eduard.bartosh@intel.com>

Fixed deb packaging issues found by lintian

The following changes have been made:
- removed libc6-dev from the list of build dependencies as it is
  build essential
- Used real Standard-Version
- Used misc:Depends macro
- Updated short description
- Added long description
- Updated package version

Change-Id: Ifa1a52d4e2f109255aa4d1ecb68dc166b57ffa42
Signed-off-by: Ed Bartosh <eduard.bartosh@intel.com>

modify pkg-confg to pkgconfig

Merge branch 'master' into tizen_2.1

Conflicts:
AUTHORS
CMakeLists.txt
debian/control
lthor.c
packaging/lthor.spec

Change-Id: Ic566170d83b2319cbcfd3db5813bd9f09e9b2534
Signed-off-by: jaehoon.you <jaehoon.you@samsung.com>

Revert "merge with master"

"merge with master" commit changed corrected code. (Prevent 35853)
So, I revert to the original state.

Reference commit : 6ea6a360fa75766b6e56545486e685ccd552ebab

--------------------------------------------------------------------
@@ -1100,7 +1100,7 @@ int test_tar_entry(struct data_src *tardata)
  return -1;
}

- if (!(chunksize <= trans_unit_size && chunksize > 0)) {
+ if ((!chunksize > 0 && chunksize <= trans_unit_size)) {
  free(chunk);
  return -1;
  }
-------------------------------------------------------------------

This reverts commit b90e7745cb6da85cbb7958ef9911bfb6c6d7fef6.

Change-Id: I1a5d0b11d7ddd73288e4dbf528331c623c99d03a

Change AUTHORS

Change-Id: Ib2991a1a6844e5a1827268f0fb4a8303c18f8269
Signed-off-by: jaehoon.you <jaehoon.you@samsung.com>

Fix and Add some error message

1. In some cases, lthor needs enough memory
(Reported by Vyacheslav Barinov in SRR)
So, notify message added.
2. fix print format about error message.

Change-Id: I42fcea8c21d4183e8909293d2101cebd19ef9dc5
Signed-off-by: jaehoon.you <jaehoon.you@samsung.com>

Add udev rules for supporting Ubuntu 12.10

In Ubuntu 12.10, modem-manager interrupt and close Tizen device.
so, add exceptional udev rules for ttyACM connection.

lthor Version 1.2 -> 1.3

Change-Id: I0c88084a8ff0f2e2f39d2b76e6568a81cf1a3e87
Signed-off-by: jaehoon.you <jaehoon.you@samsung.com>

merge with master

Tizen 2.1 base

Included udev rules into packages

Udev rules set device mode to 0666 in order to give flashing
permissions to everyone. This is handy for developers to being able to
flash devices from user account.

Change-Id: Ibb3ccb6b54c3f3c56df66ad0d17772806f622129
Signed-off-by: Ed Bartosh <eduard.bartosh@intel.com>

Updated licence information

Used Apache 2.0 licence for deb and rpm packaging.

Change-Id: I1b54726fa3a13c6752ae76eb3c9f9407708f58f4
Signed-off-by: Ed Bartosh <eduard.bartosh@intel.com>

[Prevent] fix Tizen 35853, and update authors list

Tizen 35853 : DEADCODE
original code is "assert(chunksize > 0 && chunksize <= trans_unit_size);"
(reference send_chunks() : line 125)
but chunksize already checked before line:1086 (chunksize <= 0)
then, chunksize always bigger than 0 .
so, exclamation mark is moved to another location
and conditional sentence is switched.

Change-Id: I9e5a2f998fb506fd152da856f4c9455f027207a6
Signed-off-by: jaehoon.you <jaehoon.you@samsung.com>

change license file name

This patch changes license file name which has meaning of license.

Change-Id: I832a1bfddf65aad2e1765d1b5ba5685f21e39241
Signed-off-by: Chanho Park <chanho61.park@samsung.com>

add license/author informations

This commit adds license files based on APLv2 license.

Change-Id: Ie80e337479f31660cf6298f7f3ed25fcee985356
Signed-off-by: Chanho Park <chanho61.park@samsung.com>

[Prevent] fix Tizen 36515,36516,37705 - STRING_OVERFLOW, RESOURCE_LEAK

Change-Id: I184458fd8419d4e32a9a493f2258dbbdd2637980
Signed-off-by: jaehoon.you <jaehoon.you@samsung.com>

Merge "Added option to check protocol availability"

Packaged for Debian and Ubuntu

Change-Id: I5f99117eb0bd506dd1d83d691a0b0e9c5f08c944
Signed-off-by: Ed Bartosh <eduard.bartosh@intel.com>

Packaged for rpm distros

This is a first commit regarding packaging. It implements packaging for
RPM. Debian packaging will follow.

Change-Id: Ibe45191cd3d06c387c6000c45c1b56ff820d0a87
Signed-off-by: Ed Bartosh <eduard.bartosh@intel.com>

Added option to check protocol availability

This is a first draft(not working yet) to implement a check for LTHOR protocol.
It's needed by our testing software. It should be able to first check if device
is in flashable mode and then perform flashing by running lthor <image.tar>.

To implement this feature correctly I need your help. Is there any way
in THOR protocol to inform the device that it should again wait for
handshake? If I just perform handshake and get information from the
device it will not be possible to flash it after that, because it will
be in active mode, waiting for the commands, not for the handshake.

Is it possible to contact you by email or this is the only communication
channel I can use?

Thank you,
Ed

Change-Id: Ic4cfa954885408343b5aad20fa805b2788c49ee7
Signed-off-by: Ed Bartosh <eduard.bartosh@intel.com>

initial commit of lthor

Change-Id: Ifd62a81919f2a88a22b38627dd467d53c5408823

Initial empty repository