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>

idcerpc: ask for 32kb for frag size. 256b is too small

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

check for integer wrap in query directory

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

dont segfault when trying to connect to server==NULL

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

make it possible to use alternative username for the share-enum example

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

fix correct length for error PDUs

Reported-by: Richard Sharpe <realrichardsharpe@gmail.com>
Signed-off-by: Ronnie Sahlberg <ronniesahlberg@gmail.com>

remove dependency of struct iovec from libsmb2-private.h

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

Update README and clarify some parts with kerberos/ntlmssp

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

always build with ntlmssp

Previously we would not build with ntlmssp if we had libkrb5.
Change this so that we always build with our ntlmssp implementation
and allow the user to select authentication mechanism through either the
URL argument or via an API call.

The reason for this is so that we will be able to use either krb5 or ntlmssp
from the same binary and still have the option to use sign or seal.
(the krb5 ntlmssp mech plugin does not export the session key so that
ntlm implementation can not be used for sign/seal).

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

README: document how to use/enable smb3 encryption

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

add a function to enable/disable encryption

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

make more commands use encryption is seal is enabled

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

Add plumbing to process the headers/command/data for decrypted blobs

so far we only encrypt/decrypt TreeConnect

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

split out the loops where we read header/fixed/variable parts

into its own helper function.

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

add code to decrypt smb3 transform headers and payload

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

seal: encrypt outgoing PDUs that are flagged to require encryption

Set this flag for TreeConnect for now.
We can send Encrypted TreeConnect packets but we can not yet parse
or decrypt the response.

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

initialize the prng by calling srandom on context creation

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

set SEAL flag in ntlmssp is we want encryption

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

aes128ccm: add test program

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

add aes128ccm functions

Add functions to perform aes128ccm encryption/decryption

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

When sign is used, set the ENCRYPTION capability in negprot

for protocols 3.0 and 3.02 and throw an error if the server does not support
encryption.

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

derive the encryption keys for smb3.0 and smb3.02

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

add a URL argument for seal

Add a seal argument to activate smb3 encryption.

This does not do anything yet other than record that we want to use encryption.

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

Merge pull request #97 from tguillem/android-getlogin_r-fix

Android: fix getlogin_r() dep

fix getlogin_r() dep on Android

fix getlogin_r() fallback

Merge pull request #96 from tguillem/fix-segfault-destroy

smb2_destroy_context: fix possible null-deref

smb2_destroy_context: fix possible null-deref

This could happen when the smb2_close_async() command was aborted
(smb2_service() not being called).

Merge pull request #94 from tguillem/getlogin-fix

Fix getlogin() usage

Merge pull request #95 from apuder/master

Limit output buffer size to 512 on an ESP32

Limit output buffer size to 512 on an ESP32

ntlmssp: add support for Anonymous logins

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

Fix getlogin() usage

Use the reentrant version (the getlogin() string was statically allocated and
could be overwritten on subsequent calls).

Also check for error and use "Guest" as a fallback.

Fix indent and white spaces

No functional changes.

improve error messages for readlink failures

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

add smb2_readlink example

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

fix NULL deref in readlink_cb_3

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

Add support for readlink()

Report Windows Reparse points as links
and implement readlink() to decode reparse points of the type
MS-FSCC: 2.1.2.4 Symbolic Link Reparse Data Buffer

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

New version: 3.0.0

Mon Jun 10 2019 : Version 3.0.0
- ESP32 support
- Support specifying the port number in an SMB URL.
- Add creation time to smb_stat
- Abort all commands when the context is destroyed.
- Free all file and directory handles when the context is destroyed.
- Fix handling of O_TRUNC
- Add more error codes.
- Add support for SMB2_IOCTL
- Handle DCE/RPC fragment reassembly for IOCTL().

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

Merge pull request #91 from apuder/esp

Add instructions for ESP32 and small fix

Make libsmb2's config.h private

Add instructions for the ESP32

Merge pull request #90 from apuder/esp

Add support for the ESP32 microcontroller

Add support for the ESP32 microcontroller