Merge branch 'master' of github.com:sahlberg/libsmb2

dcerpc: fix handling of pointers for NetShareGetInfo ndr64

our pointer alignment was broken

Signed-off-by: Ronnie Sahlberg <ronniesahlberg@gmail.com>

Added connection timeout for sync interface, now if not 'is_connected' after 'timeout' seconds, operation is aborted regardless of status

Merge pull request #149 from timmeh87/master

Improved timeout for sync interface when connection does not exist

dcerpc: rework dcerpc and try to layer it a little better vs smb2

Signed-off-by: Ronnie Sahlberg <ronniesahlberg@gmail.com>

Added connection timeout for sync interface, now if not 'is_connected' after 'timeout' seconds, operation is aborted regardless of status

Merge pull request #147 from timmeh87/master

winsock error message / null pointer checks

Added windows specific error message if winsock is not init
added null pointer checks to some of the core api functions to improve reliability

smb2-cat-sync: improve the example read loop

Signed-off-by: Ronnie Sahlberg <ronniesahlberg@gmail.com>

smb2-cat-sync: try to use very large reads

Signed-off-by: Ronnie Sahlberg <ronniesahlberg@gmail.com>

pread/pwrite: make sure to always update the number of credits we need

Signed-off-by: Ronnie Sahlberg <ronniesahlberg@gmail.com>

Add support for timeout

Signed-off-by: Ronnie Sahlberg <ronniesahlberg@gmail.com>

sync: don't access freed memory for cancelled PDUs

If we fail wait_for_reply in the sync API it means that the the underlying
SMB2 PDU might still be in fligth and our sync callback will be invoked later.

Solve this by not freeing the cb_data structure on wait_for_reply failures
and instead just flag the status as CANCELLED.
Later when the callback is invoked, if status is CANCELLED it means we should
just free cb_data and return.

Signed-off-by: Ronnie Sahlberg <ronniesahlberg@gmail.com>

sync: don't use cb structures allocated on the stack

Signed-off-by: Ronnie Sahlberg <ronniesahlberg@gmail.com>

Revert "We can not access cb_data when status is CANCELLED"

This reverts commit 5c4d6db14ce4048b7d08d9af3307022ba4992833.

Merge pull request #139 from amandeepgautam/master

return appropriate error codes on failure

Add a URL argument and API to enforce signing

Signed-off-by: Ronnie Sahlberg <ronniesahlberg@gmail.com>

return appropriate error codes on failure

with current change one can have same error handling logic for callbacks and
function return values. '-1' corresponds to permission denied which
makes it problematic.

Merge pull request #138 from amandeepgautam/master

fix regression from commit: "extend error codes (from samba)"

fix regression from commit: "extend error codes (from samba)"

Merge pull request #136 from amandeepgautam/master

add const for filename

add const for filename

dcerpc: the srvsvc_interface belong in dcerpc-srvsvc.c not dcerpc.c

Signed-off-by: Ronnie Sahlberg <ronniesahlberg@gmail.com>

dcerpc: add example of NetShareGetInfo

Signed-off-by: Ronnie Sahlberg <ronniesahlberg@gmail.com>

dcerpc: make it possible to control le/be from url arguments for testing

Signed-off-by: Ronnie Sahlberg <ronniesahlberg@gmail.com>

dcerpc: make it possible to select ndr transfer syntax from url for testing

Signed-off-by: Ronnie Sahlberg <ronniesahlberg@gmail.com>

dcerpc: fix top level ref pointer bug.

Signed-off-by: Ronnie Sahlberg <ronniesahlberg@gmail.com>

dcerpc: add comment that NDR64 is only available for LE encodings

Signed-off-by: Ronnie Sahlberg <ronniesahlberg@gmail.com>

dcerpc: we need portable-endian.h for some platforms

Signed-off-by: Ronnie Sahlberg <ronniesahlberg@gmail.com>

dcerpc: support sending dcerpc pdus in big-endian mode

Signed-off-by: Ronnie Sahlberg <ronniesahlberg@gmail.com>

dcerpc: track the data representation in the dcerpc context

Signed-off-by: Ronnie Sahlberg <ronniesahlberg@gmail.com>

We can not access cb_data when status is CANCELLED

since the pointer may no longer be valid.

Reported-by: hgourvest@progdigy.com
Signed-off-by: Ronnie Sahlberg <ronniesahlberg@gmail.com>

Fix cast in dcerpc.c

Signed-off-by: Ronnie Sahlberg <ronniesahlberg@gmail.com>

IPv6 is available on all platforms. No need to check for it

We weren't even checking for this in configure.ac anyway.

Signed-off-by: Ronnie Sahlberg <ronniesahlberg@gmail.com>

Merge pull request #130 from amandeepgautam/master

extend error codes (from samba)

Add callbacks to notify application of fd or event changes

Add callbacks to allow libsmb2 to call back into the application to
inform when the filedescriptor to wait on changes or when the set
ot events (POLLIN/POLLOUT) changes.

This allows better integration with more sophisticated eventsystems
such as those built on epoll() than the older way libsmb2 provided.

Signed-off-by: Ronnie Sahlberg <ronniesahlberg@gmail.com>

Merge pull request #132 from 76716C86/master

fix __cplusplus guards

fix __cplusplus guards

* Some functions weren't included in the __cplusplus scope which caused
the compiler to mangle those names which made it impossible for the
linker to find the proper names.

* Properly close all the __cplusplus scopes. Some of them were left
opened and caused weird errors when compiling c++ code.

extend error codes (from samba)

Merge pull request #128 from amandeepgautam/master

add const qualifier for smb write related code

add const qualifier for smb write related code

Merge pull request #124 from amandeepgautam/master

do not clamp read/write request to 60KB

do not clamp read/write request to 60KB

Merge pull request #123 from amandeepgautam/master

evaluate smb status before call to krb5 lib

update readme to say that signing is supported with ntlm plugin

evaluate smb status before call to krb5 lib

This leads to better error messages. For example, for incorrect pass we
have a message like following:
Session setup failed with (0xc000006d) STATUS_LOGON_FAILURE (-111)
with current change instead of
gss_init_sec_context: (Invalid token was supplied, Unknown error) (-1)

fix SIGBUS in smb2_set_uint64

Signed-off-by: Ronnie Sahlberg <ronniesahlberg@gmail.com>

libsmb2: don't call dcerpc_free_pdu to avoid use-after-free

 ret = dcerpc_decode_pdu(dce, pdu, &iov);
 if (ret < 0) {
 pdu->cb(dce, -EINVAL, NULL, pdu->cb_data); -> use after free.
 smb2_free_data(dce->smb2, rep->output);
 dcerpc_free_pdu(dce, pdu);  -> re-free here.
 return;
 }

Signed-off-by: Namjae Jeon <linkinjeon@gmail.com>

libsmb2: fix potential memory leak

Signed-off-by: Namjae Jeon <linkinjeon@gmail.com>

libsmb2: alloc_hint with unint32 type never less than zero

unsigned type alloc_hint could not be negative.

Signed-off-by: Namjae Jeon <linkinjeon@gmail.com>

libsmb2: fix use-after-free of c_data->utf8_unc

freed c_data->utf8_unc by free_c_data could be used.

Signed-off-by: Namjae Jeon <linkinjeon@gmail.com>

Use calloc where appropriate

Saves around 250 bytes with -Oz

negotiate protocol: some servers do not return a blob in neg prot responses

Signed-off-by: Ronnie Sahlberg <ronniesahlberg@gmail.com>

Merge pull request #118 from tguillem/more-errnos

More errnos

smb2_connect_share_async: don't return ENOMEM if connect fails

smb2_connect_async: convert getaddrinfo to errno

smb2_connect_async: return errno

Merge pull request #113 from ePirat/configure-cleanup

Configure cleanup

Use AS_IF instead of shell if statements

This is more consistent with the rest of the file

Use GCC variable instead of ac_cv_prog_gcc

This is documented in the AC_PROG_CC documentation and seems a bit
cleaner than to use the "internal" caching variable. And it is easier
to read.

Remove unused header checks

None of the headers checked by these checks are used in the project.

Cleanup comments

A lot of comments were duplicated as both autoconf and shell
comments, which seems unnecessary.
Additionally slight rewording of some comments.

Merge pull request #112 from ePirat/ci-run-prs

Trigger CI builds for pull requests

Merge pull request #111 from ePirat/add-ci-windows

Add windows build to CI

Merge pull request #110 from ePirat/fix-windows-msvc

Various MSVC build fixes

Fix indentation

Add missing quotes

While these are not all needed it is much easier to be consistent and
quote, so that when editing later to not accidentally forget to quote
a value that did not need quoting before.

Do not use deprecated AC_HELP_STRING

Remove explicit PKG_PROG_PKG_CONFIG call

This will be called implicitly when needed

Set minimum supported autoconf version to 2.58

The file is already using things only available since 2.58, not 2.50

Check for AM_PROG_AR instead of defining to nothing

Set needed automake version to 1.11

This version should be old enough and gives us an opportunity to know
what is actually the lowest supported version, erroring out with a
helpful error message if someone tries to use an older version than
that, instead of obscure errors due to missing macros.

Simplify suppression of default CFLAGS

This is the way autoconf documents this should be done and much
simpler than before.

Add config.h.in~ to gitignore

Trigger CI builds for pull requests

Add windows build to CI

Various MSVC build fixes

Fix #101

Merge pull request #108 from ePirat/add-ci

Add GitHub Actions CI

Merge pull request #107 from ePirat/fix-distcheck

Fix autotools distcheck

Add GitHub Actions CI

Remove check for popt and fix conditional directory

Using an automake conditional here allows automake to correctly
always add examples sources to the dist tarballs, even when disabled.

Fixes `make distcheck`

Add missing headers files to Makefile.am

This fixes `make distcheck` and produces correct
tarballs with `make dist`.

Merge pull request #105 from ePirat/remove-popt-dep

Remove unused popt dependency for examples

Merge pull request #106 from ePirat/fix-macos-compilation

aes128ccm: Add missing portable-endian header

Merge pull request #104 from ePirat/add-error-code

errors: Add SMB2_STATUS_BAD_NETWORK_PATH

errors: Add SMB2_STATUS_BAD_NETWORK_PATH

This error code is returned by macOS SMB implementation when not
specifying the share to which to connect.

aes128ccm: Add missing portable-endian header

This fixes the compilation on macOS and other platforms that
do not have htobe* functions.

Remove unused popt dependency for examples

The examples have a dependency on popt even though that library is
not used by any examples, making it unnecessary.

Merge pull request #102 from tguillem/smb2-password-fix

Don't override password set via smb2_set_password()

Revert "fix correct length for error PDUs"

This reverts commit 938cc11fabfb7c31e88ffcd2d04fce859664a9e1.

Some servers (win10 for anonymous/guest logins) actually violate the spec
here and does NOT provide the extra byte.

don't override password set via smb2_set_password()

Regression from bf5c12d7072d3a81438ffee67e7308077bfa3adb

This was impossible for an user to provide a password via smb2_set_password()
since it was overridden from smb2_connect_share_async(). Indeed,
smb2_set_password_from_file() is called from smb2_set_user(), that is called
from smb2_connect_share_async().

check fopen return value

krb5: ignore errors if SMB2 layer returned success

Signed-off-by: Ronnie Sahlberg <ronniesahlberg@gmail.com>

change bitfield from int:1 to uint8_t:1 to make lgtm.com happy

Signed-off-by: Ronnie Sahlberg <ronniesahlberg@gmail.com>

Add header guards to make lgtm.com happy

Signed-off-by: Ronnie Sahlberg <ronniesahlberg@gmail.com>

we don't need SHA512 either

This all now shaves off 5-6kb of text size !!!

Signed-off-by: Ronnie Sahlberg <ronniesahlberg@gmail.com>

we don't need SHA384

Signed-off-by: Ronnie Sahlberg <ronniesahlberg@gmail.com>

we dont need sha1

Signed-off-by: Ronnie Sahlberg <ronniesahlberg@gmail.com>

We don't need SHA224

Signed-off-by: Ronnie Sahlberg <ronniesahlberg@gmail.com>

aes: we do not need the CBC version of AES

Signed-off-by: Ronnie Sahlberg <ronniesahlberg@gmail.com>